-
Notifications
You must be signed in to change notification settings - Fork 230
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
Make printed structures slightly more readable #2910
base: develop
Are you sure you want to change the base?
Conversation
I'm not at my dev computer to test this right now, but can you increase the max depth a bit so the RGBA() and RANGE() values get printed out in the example script? They were in there to test the case of something that does have a ToString() defined but is also Serializable so it has Dump logic, to see if the Dump printout ends up overriding the ToString printout like it does with the current code. (i.e. the problem in the current implemention, prior to either PR, is that instead of seeing this as the value in the list:
it showed this:
(Which led to the strange effect that the 8th element when you |
Unrelated note - how are you running that in the VAB? |
develop...thexa4:feature/ksc-mainframe (has quite a lot of changes and isn't well tested, didn't want to create a PR for it yet) |
TODO:
|
Initial implementation doesn't crash instantly, testing with it now. |
Could be a possible solution for #2894
Example output with the same script as #2909: