Skip to content
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

Use csv table #30

Open
forsyth2 opened this issue Nov 1, 2023 · 1 comment
Open

Use csv table #30

forsyth2 opened this issue Nov 1, 2023 · 1 comment
Labels
documentation Files in `docs` modified

Comments

@forsyth2
Copy link
Collaborator

forsyth2 commented Nov 1, 2023

https://stackoverflow.com/questions/26609816/some-online-tool-or-automation-plugin-for-sublimetext-for-generating-sphinx-rst describes how to define the rst table using csv. We should change the implementation of generate_tables.py to do that since the rst tables are rather unwieldy.

In the current implementation, a column has to be exactly the same length in every single row, or else there will be a malformed table. For that matter, any problem with the table just shows a generic "malformed table" error which is of no use when debugging.

@forsyth2 forsyth2 added the documentation Files in `docs` modified label Nov 1, 2023
@chengzhuzhang
Copy link
Collaborator

Thanks for fixing my PR creating this issue which is a good potential solution. @TonyB9000 mentioned that he's working on a PR to streamline the simulation data table. We can probably implement some csv based solution there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Files in `docs` modified
Projects
None yet
Development

No branches or pull requests

2 participants