Skip to content

Commit

Permalink
removed checks for OpenMDAO older than the oldest supported version
Browse files Browse the repository at this point in the history
  • Loading branch information
robfalck committed Aug 14, 2023
1 parent 5817db0 commit 405d4a0
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,6 @@ def compute(self, inputs, outputs):
@use_tempdirs
class TestConnectControlToParameter(unittest.TestCase):

@unittest.skipIf(om_version < (3, 4, 1) or (om_version == (3, 4, 1) and om_dev_version),
'test requires OpenMDAO >= 3.4.1')
@require_pyoptsparse(optimizer='SLSQP')
def test_connect_control_to_parameter(self):
""" Test that the final value of a control in one phase can be connected as the value
Expand Down
5 changes: 1 addition & 4 deletions dymos/phase/phase.py
Original file line number Diff line number Diff line change
Expand Up @@ -2312,10 +2312,7 @@ def initialize_values_from_phase(self, prob, from_phase, phase_path='', skip_par

# We use this private function to grab the correctly sized variable from the
# auto_ivc source.
if om_version < (3, 4, 1):
val = phs.get_val(f'parameters:{name}', units=units)[0, ...]
else:
val = phs.get_val(f'parameters:{name}', units=units)
val = phs.get_val(f'parameters:{name}', units=units)

if phase_path:
prob_path = f'{phase_path}.{self.name}.parameters:{name}'
Expand Down
1 change: 0 additions & 1 deletion dymos/test/test_load_case.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ def setup_problem(trans=dm.GaussLobatto(num_segments=10), polynomial_control=Fal
return p


@unittest.skipIf(om_version <= (2, 9, 0), 'load_case requires an OpenMDAO version later than 2.9.0')
@use_tempdirs
class TestLoadCase(unittest.TestCase):

Expand Down
1 change: 0 additions & 1 deletion dymos/test/test_run_problem.py
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,6 @@ def test_illegal_simulate_kwargs(self):
'Key "case_prefix" was found in simulate_kwargs but should instead by provided by '
'the argument "case_prefix", not part of the simulate_kwargs dictionary.')

@unittest.skipIf(om_version < (3, 18, 0), 'test requires OpenMDAO >= 3.18.01')
@require_pyoptsparse(optimizer='SLSQP')
def test_run_brachistochrone_problem_refine_case_driver_case_prefix(self):
p = om.Problem(model=om.Group())
Expand Down

0 comments on commit 405d4a0

Please sign in to comment.