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

Updated dymos to handle the new OpenMDAO distributed I/O approach #597

Merged
merged 4 commits into from
May 19, 2021

Conversation

robfalck
Copy link
Contributor

@robfalck robfalck commented May 14, 2021

Summary

Updated vanderpol MPI case to follow the new OpenMDAO way of doing serial/distributed I/O rather than entire components.

Trajectory no longer assigns solvers to phases.

Phases will now only assign a solver if it is needed by solve_segments and if the exisiting one is NonlinearRunOnce or LinearRunOnce. This should allow users to override the default solver before setup, but also make turning solve_segments on and off easy in most cases.

Related Issues

Status

  • Ready for merge

Backwards incompatibilities

None

New Dependencies

None

robfalck added 2 commits May 14, 2021 16:33
…rial/distributed I/O rather than entire components.

Trajectory no longer assigns solvers to phases.
Phases will now only assign a solver if it is needed by solve_segments and if the exisiting one is NonlinearRunOnce or LinearRunOnce.
This should allow users to override the default solver before setup, but also make turning solve_segments on and off easy in most cases.
@coveralls
Copy link

coveralls commented May 14, 2021

Coverage Status

Coverage decreased (-0.1%) to 96.619% when pulling 1273551 on robfalck:vanderpol_mpi_update into d4e68bd on OpenMDAO:master.

@robfalck robfalck requested a review from Kenneth-T-Moore May 18, 2021 15:41
@robfalck robfalck merged commit d959ecc into OpenMDAO:master May 19, 2021
@robfalck robfalck deleted the vanderpol_mpi_update branch August 25, 2022 15:53
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.

Update distributed vanderpol test to support newer versions of OpenMDAO.
3 participants