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

Feature parity for DG code #190

Merged
merged 9 commits into from
Jun 18, 2024
Merged

Feature parity for DG code #190

merged 9 commits into from
Jun 18, 2024

Conversation

jbreue16
Copy link
Contributor

@jbreue16 jbreue16 commented Apr 22, 2024

Feature parity for the DG models, which were implemented in #110

To-do

@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch from a5705c3 to 556c3e4 Compare April 22, 2024 10:19
@jbreue16 jbreue16 mentioned this pull request Apr 22, 2024
6 tasks
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch from 556c3e4 to c3ff5db Compare April 22, 2024 10:38
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch 2 times, most recently from a2a0339 to 24e9873 Compare May 5, 2024 11:40
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch from 24e9873 to 2ceab87 Compare May 23, 2024 15:22
@jbreue16 jbreue16 added this to the v5.0.0 "CADET-Core" milestone Jun 2, 2024
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch 4 times, most recently from 589cfc7 to 3061e31 Compare June 4, 2024 10:21
@jbreue16 jbreue16 requested a review from sleweke June 4, 2024 10:32
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch 2 times, most recently from 8ba85b6 to f04df12 Compare June 4, 2024 11:17
jbreue16 and others added 9 commits June 18, 2024 11:02
* Change DG unit interfaces from unit_type suffix to discretization specification

* Infer DG disc from NCOL and NPAR and set default values

* Rename DG element fields to nelem and par_nelem

* Fix test interfaces, i.e. adjust to createUnitOperation function

* Improve parameter interface DG units (according to #177)

Co-authored-by: sleweke <[email protected]>
* LRM inlet DOF treatment

* LRMP flux Jacobian analytical and AD fix

* GRM allocation of deltaR

* Clean up

* Fix bug in LRMP DG sensitivity init
Add tolerance option to AD jacobian comparison tests

Add relative tolerances to FV numerical reference tests
@jbreue16 jbreue16 force-pushed the feature/dg_feature_parity branch from 4991086 to b3b6b38 Compare June 18, 2024 09:03
@jbreue16 jbreue16 requested a review from AntoniaBerger June 18, 2024 09:42
@jbreue16 jbreue16 merged commit 5a696d5 into master Jun 18, 2024
3 checks passed
@jbreue16 jbreue16 self-assigned this Jul 24, 2024
@jbreue16 jbreue16 deleted the feature/dg_feature_parity branch November 12, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants