-
Notifications
You must be signed in to change notification settings - Fork 12
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
reworking and adjusting the output table function #116
base: main
Are you sure you want to change the base?
Conversation
It would be nice to improve
|
…for new creeplaws
Are you still planning to work on this? I am wondering whether it would makes sense to move this out to an external package, something like |
I do not have a lot of time at the moment, but I will finish this pull request once I have more time again. |
I think it make sense to move it out. It is not really needed inside GP. And I don't think it will change the current code structure, since its already a submodule, just need to update a few lines of code here and there. |
I'm fine with moving this to a separate package |
I want to improve the output tables functions.
E.g. the old output tables function uses a dict inside a function to find the parameter description/name from the corresponding variable. I want to try to improve this to be more independent of the hard coded dict and descriptions.
Additionally, the tables are adjusted for the new rheologies that were added with #99.