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

automated convergence studies #368

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

richardschu
Copy link
Contributor

split-off from #366
automated convergence study analysis taking files generated by ErrorCalculator, now writing the files in a programmatic way: in the first call (when originally the files were only flushed), we now give a new run_id if a file with the same filename_base exists.

This allows splitting up the studies in various parts, or just displaying all the error in one go after a single potentially small run for easier extraction of the data. Since the ErrorCalculators are in utilities, one just needs to adapt the individual solver.h files instead of postprocessing individual log files. After all, the solver.h were designed to run the studies already.

If no files or no directory exists, we print a message; if only a single file exists, there is a specific message as well, and if there are multiple files found, a convergence study is populated and printed.

@nfehn nfehn added the new feature New feature is implemented or requested label Mar 10, 2023
@richardschu
Copy link
Contributor Author

I have to check the tests, the DIFF fails, I will do this tomorrow ; )

@richardschu richardschu force-pushed the automated_convergence_analysis branch from 67f5fb0 to 5279d57 Compare March 12, 2023 10:02
@richardschu richardschu force-pushed the automated_convergence_analysis branch from 7e733fb to 51f3b86 Compare March 12, 2023 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature is implemented or requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants