search

IAMconsortium / pyam

Analysis & visualization of energy & climate scenarios
https://pyam-iamc.readthedocs.io/
Apache License 2.0
236 stars 120 forks source link

Use context manager for reading Excel file #818

Closed phackstock closed 9 months ago

phackstock commented 9 months ago

Please confirm that this PR has done the following:

Description of PR

Closes #817. Use a context manager to ensure that an Excel file is closed after reading.

FYI @adrivinca

codecov[bot] commented 9 months ago

Codecov Report

Attention: Patch coverage is 90.90909% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 94.8%. Comparing base (0762055) to head (e503e65).

Files Patch % Lines
pyam/utils.py 90.9% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #818 +/- ## ===================================== Coverage 94.8% 94.8% ===================================== Files 64 64 Lines 6089 6089 ===================================== Hits 5775 5775 Misses 314 314 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

phackstock commented 9 months ago

Pinned ixmp4==0.6.0 after a test broke when using ixmp4 version 0.7.0.