-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
improvements #2
Comments
Thanks for the input! For the color function there's no no need to place more utils, rounds (more cpu) and take down the "minimum" "medium" and "max" threshold warning. For color conventions, I do endorse the use of hexs and web colors thru names. You can still use rgb, but it's a bit, duhhhh. (unless u need to do a transparency, and in that case, you should use an argb). For now I will not make different containers in one hub. We have character limit on the names (30), and it will be to damn confuse to accomplish that. Last version brought the capacity of multiple "standalone" containers with the same ore, but but different sizes within one container hub is very tricky. CH_1L_3M_4S_IRON ?? Don't have space left... For the temp variable in the first quote, i could make it local. It will prevent future unpleasantness if I reuse that name. Thanks! |
Yes, the character limit does not allow to type everything. Thank you! P.S. if you add a formatted version of your code, it will be easier for visitors to give you feedback. Not everyone wants to copy this to the game or parse it. |
Alternatively, you can use the capacity instead: CH_196.0_IRON or CH_IRON_196.0 |
Added the locals on the core code. |
Please check all cycles and local variables across the code.
Please see comments below
Improvement to color the line:
This code cannot work with different containers in one hub.
Please consider specifying the volume/capacity in the name of the hub.
The text was updated successfully, but these errors were encountered: