-
Notifications
You must be signed in to change notification settings - Fork 5
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
182 surface reaction example #356
Conversation
@dwfncar could you move this to the analytical tests? That's where the other tests like this are |
Yes, hadn't ever looked at that file, so wasn't sure where this belonged. Will add the EXPECT_NEAR assertions as well. |
…o analytical_policy.hpp.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #356 +/- ##
=======================================
Coverage 96.07% 96.07%
=======================================
Files 37 37
Lines 2852 2852
=======================================
Hits 2740 2740
Misses 112 112 ☔ View full report in Codecov by Sentry. |
Added TEST(AnalyticalExamplesRosenbrock, SurfaceRxn) and JIT version. PR ready for review. |
Waiting for #326 merge, to add the new state printer header and print state methods.
Should this test be made into a full science example?
Setup was copied from https://github.com/open-atmos/camp/blob/main/test/unit_rxn_data/test_rxn_surface.F90
closes #182