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

Transient #58

Open
wants to merge 4 commits into
base: devel
Choose a base branch
from
Open

Transient #58

wants to merge 4 commits into from

Conversation

mguddati
Copy link
Contributor

Transient inversion: Introduced an example with gradient check,
modifying the code to make the gradient check pass (by making it
consistent with discretize then differentiate strategy).

Added Hessian inversion with associated test case, and verified
that it converges as expected.

Also added code functionality to facilitate consistent gradient
inverse for time steppers beyond implicit Euler.

@mguddati mguddati force-pushed the transient branch 2 times, most recently from 330dcf1 to 3422655 Compare September 13, 2022 15:14
@moosebuild
Copy link

Job Precheck on 3422655 wanted to post the following:

Your code requires style changes.

A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:

curl -s https://mooseframework.inl.gov/isopod/docs/PRs/58/clang_format/style.patch | git apply -v

Alternatively, with your repository up to date and in the top level of your repository:

git clang-format fba1b6a7cfda595b278990d04b41d9843425171c

@moosebuild
Copy link

moosebuild commented Sep 13, 2022

Job Documentation on 5dc5b83 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link

moosebuild commented Sep 13, 2022

Job Coverage on 5dc5b83 wanted to post the following:

Coverage

ad3bea #58 5dc5b8
Total Total +/- New
Rate 87.37% 87.53% +0.16% 100.00%
Hits 1003 1004 +1 4
Misses 145 143 -2 0

Diff coverage report

Full coverage report

This comment will be updated on new commits.

@mguddati mguddati force-pushed the transient branch 3 times, most recently from b815b37 to 7c67c68 Compare September 20, 2022 16:23
@lynnmunday lynnmunday force-pushed the transient branch 2 times, most recently from bd44074 to be2d605 Compare September 23, 2022 00:00
Murthy and others added 4 commits September 22, 2022 18:02
modifying the code to make the gradient check pass (by making it
consistent with discretize then differentiate strategy).

Added Hessian inversion with associated test case, and verified
that it converges as expected.

Also added code functionality to facilitate consistent gradient
inverse for time steppers beyond implicit Euler.

ref #42
…ut also for ActuallyExplicitEuler.

                     Made the end time consistent to the formulation and adaptable based on
                     time integration scheme.
                     Added small test case that is not heavy to test both time integrators
                     Made other necessary test changes
… timesteps. Added diff forward output file instead of optimization reporter ref #42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants