More plots for the RBC paper#594
Conversation
| assert not np.allclose(NuI, ref_data['Nu']['V']) | ||
| assert np.allclose(tI, ref_data['t']) | ||
|
|
||
|
|
There was a problem hiding this comment.
Why are there no decorators here to flag the tests?
There was a problem hiding this comment.
The project tests are all run in the project specific environment. So in the current CI, environment markers for project tests have no impact.
The GPU project is tested on JUWELS using the environment we set up there, so it's not even using micromamba.
Now that I think about it, it may make sense to put the 3D RBC stuff in its own project so the tests can be run on GitHub and CPUs. For testing purposes, CPUs should be fine.
What do you think? New project or tests on JUWELS?
There was a problem hiding this comment.
Both, since we need the GPU tests?
There was a problem hiding this comment.
I don't think we really need GPU tests for this. We have tests that the GPU port of spectral discretizations generally works, so CPU only testing for this should be fine. While a test of course doesn't hurt, I think the work involved here would not be worth while.
I'll set up a new project then and do CPU tests.
There was a problem hiding this comment.
OK! What should we do with this PR, then?
There was a problem hiding this comment.
I suggest we merge it and then I do the move in a separate PR to keep the diff small.
2 participants
This PR includes a plot for the spectrum and one for the order of accuracy for various simulation schemes. Functions that compute stuff are tested. Functions that only plot stuff or that only supply parameters for plotting are excluded from the coverage report.