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

Time integrator #5463

Merged
merged 2 commits into from Jul 29, 2015
Merged

Time integrator #5463

merged 2 commits into from Jul 29, 2015

Conversation

permcody
Copy link
Member

@danielru - I was able to keep most of your work so thanks for the contribution!

A couple of things I had to change

  • I added logic to the Action you originally added to make it work
  • Then for error checking I had to swap the order of the tasks so that we don't get into a situation where multiple integrators get added silently causing confusing behavior.
  • Finally I put in a few more tests and undid your modification to Example 6

@moosebuild
Copy link
Contributor

Results of testing ad7cb1f using moose_PR_pre_check recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.org/view_job/20542

@moosebuild
Copy link
Contributor

Results of testing ad7cb1f using moose_PR_test recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.org/view_job/20543

@moosebuild
Copy link
Contributor

Results of testing ad7cb1f using moose_PR_app_tests recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.org/view_job/20545

@moosebuild
Copy link
Contributor

Results of testing ad7cb1f using moose_PR_test_dbg recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.org/view_job/20544

@moosebuild
Copy link
Contributor

Results of testing ad7cb1f using moose_PR_modules_dbg recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.org/view_job/20546

jwpeterson added a commit that referenced this pull request Jul 29, 2015
@jwpeterson jwpeterson merged commit 9c27f69 into idaholab:devel Jul 29, 2015
@permcody permcody deleted the time_integrator branch June 7, 2016 20:19
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.

None yet

3 participants