Skip to content

Remove shooting method#808

Merged
jkirk5 merged 11 commits intoOpenMDAO:mainfrom
kaushikponnapalli:remove_simupy
Jul 18, 2025
Merged

Remove shooting method#808
jkirk5 merged 11 commits intoOpenMDAO:mainfrom
kaushikponnapalli:remove_simupy

Conversation

@kaushikponnapalli
Copy link
Copy Markdown
Contributor

Summary

Removed all references to shooting, simupy/SGM and analysis scheme. Now Collocation is the only option available

Related Issues

  • Resolves #

Backwards incompatibilities

Shooting no longer works

New Dependencies

None

@kaushikponnapalli kaushikponnapalli changed the title Remove shooting method [Draft] Remove shooting method Jun 30, 2025
prop_group = om.Group()
else:
prop_group = self
prop_group = self
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since there isn't an alternate for prop_group anymore I think we can skip this, just use self everywhere now

@jkirk5
Copy link
Copy Markdown
Contributor

jkirk5 commented Jul 2, 2025

Looks like there's one last mention of AnalysisScheme in coupled_aircraft_mission_optimization.ipynb which is causing the docs failure

@xjjiang
Copy link
Copy Markdown
Contributor

xjjiang commented Jul 5, 2025

Looks like there's one last mention of AnalysisScheme in coupled_aircraft_mission_optimization.ipynb which is causing the docs failure

There are more places in Docs.

@kaushikponnapalli kaushikponnapalli changed the title [Draft] Remove shooting method Remove shooting method Jul 11, 2025
@jkirk5 jkirk5 added this pull request to the merge queue Jul 16, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 16, 2025
@jkirk5 jkirk5 added this pull request to the merge queue Jul 18, 2025
Merged via the queue into OpenMDAO:main with commit e4ff7a3 Jul 18, 2025
6 checks passed
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.

4 participants