Merge pull request #46 from Missing-Tech/master

Close yaml files after opening
junzis · Apr 15, 2024 · 8559381 · 8559381
2 parents 8e6a2bc + d32058d
commit 8559381
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/openap/drag.py b/openap/drag.py
@@ -65,7 +65,8 @@ def dragpolar(self):
                 raise RuntimeError(f"Drag polar for {self.ac} not avaiable in OpenAP.")
 
         f = dir_dragpolar + ac + ".yml"
-        dragpolar = yaml.safe_load(open(f))
+        with open(f, "r") as file:
+            dragpolar = yaml.safe_load(file.read())
         return dragpolar
 
     @ndarrayconvert

diff --git a/openap/prop.py b/openap/prop.py
@@ -56,7 +56,8 @@ def aircraft(ac, use_synonym=False, **kwargs):
             raise RuntimeError(f"Aircraft {ac} not avaiable in OpenAP.")
 
     f = files[0]
-    acdict = yaml.safe_load(open(f))
+    with open(f, "r") as file:
+        acdict = yaml.safe_load(file.read())
 
     return acdict