Merge pull request #3202 from eytanadler/fix_hotstart

Fix pyOptSparse `hotstart_file` `@property` method
OpenMDAO · Apr 24, 2024 · 3eab709 · 3eab709
2 parents b172b21 + 1394b59
commit 3eab709
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/openmdao/drivers/pyoptsparse_driver.py b/openmdao/drivers/pyoptsparse_driver.py
@@ -306,7 +306,7 @@ def hotstart_file(self):
         """
         warn_deprecation("The 'hotstart_file' attribute is deprecated. "
                          "Use the 'hotstart_file' option instead.")
-        return self.options['hist_file']
+        return self.options['hotstart_file']
 
     @hotstart_file.setter
     def hotstart_file(self, file_name):

diff --git a/openmdao/drivers/tests/test_pyoptsparse_driver.py b/openmdao/drivers/tests/test_pyoptsparse_driver.py
@@ -2816,7 +2816,7 @@ def test_hist_file_hotstart_deprecated(self):
         with assert_warning(OMDeprecationWarning, msg):
             driver.hist_file = filename
         with assert_warning(OMDeprecationWarning, msg):
-            driver.hist_file
+            self.assertEqual(driver.hist_file, filename)
         self.assertEqual(driver.options['hist_file'], filename)
 
         prob.setup()
@@ -2833,7 +2833,7 @@ def test_hist_file_hotstart_deprecated(self):
         with assert_warning(OMDeprecationWarning, msg):
             driver.hotstart_file = filename
         with assert_warning(OMDeprecationWarning, msg):
-            driver.hotstart_file
+            self.assertEqual(driver.hotstart_file, filename)
         self.assertEqual(driver.options['hotstart_file'], filename)
 
         prob.setup()