-
Notifications
You must be signed in to change notification settings - Fork 97
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
Convert text dumps to json format #28
Comments
This was partially resolved with #22! |
Yes, I realized in doing it that the issue with |
That seems fine with me! I would, however, potentially consider raising a separate issue for reorganizing the |
Yes, I think that's fair. That might even be a more accurate description of this issue... |
This was originally about the component table format, right? Given that we're now using a csv format for those tables, should we close this? |
Agreed ! Yes, this was still open if we wanted to use JSON instead, but that's overkill at this point I think. Closing :) |
Changing a few np.integer to np.int32
Comes from #24 (comment)
Right now,
csstepdata
is generated by evaluating variables and saving out the result to a text file. Something similar happens inseldict
, although there the result is returned as a dictionary.It would be great to instead
zip
together a list of the variables with descriptive identifiers. Then, if the dictionary is being saved out, we could usejson.dumps()
to write out a structured instead of plain text file.The text was updated successfully, but these errors were encountered: