Skip to content

Commit

Permalink
Fixed a bug with min time climb where number of nodes was specified i…
Browse files Browse the repository at this point in the history
…ncorrectly and updated new load case test
  • Loading branch information
kaushikponnapalli committed Jul 30, 2024
1 parent 1e636ad commit 47e59f7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ def min_time_climb(optimizer='SLSQP', num_seg=3, transcription='gauss-lobatto',
elif transcription == 'radau-ps':
tx = dm.Radau(num_segments=num_seg, order=transcription_order)
elif transcription == 'birkhoff':
tx = dm.Birkhoff(order=transcription_order)
tx = dm.Birkhoff(num_nodes=transcription_order)

traj = dm.Trajectory()

Expand Down
2 changes: 1 addition & 1 deletion dymos/test/test_load_case.py
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ def test_load_case_radau_to_birkhoff(self):
case = om.CaseReader('dymos_solution.db').get_case('final')

# create a problem with a different transcription with a different number of variables
q = setup_problem(dm.Birkhoff(grid=dm.BirkhoffGrid(num_nodes=50)))
q = setup_problem(dm.Birkhoff(num_nodes=50))

# Fill q with junk so that we can be sure load_case worked
q['phase0.t_initial'] = -88
Expand Down

0 comments on commit 47e59f7

Please sign in to comment.