Closed bouweandela closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 94.51%. Comparing base (
2fc8761
) to head (04d522e
). Report is 41 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@chrisbillowsMO if it is still possible, this bugfix would also be nice to have in the release.
@chrisbillowsMO if it is still possible, this bugfix would also be nice to have in the release.
@bouweandela Shouldn't be a problem. Have added to our checklist.
Many thanks @chrisbillowsMO - merge at will ๐บ
Description
Due to https://github.com/psf/requests/issues/5536, downloading files from ESGF loaded entire files into memory, leading to potentially high memory use when downloading large files. This pull request avoids the issue.
Before you get started
Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the ๐ Technical or ๐งช Scientific review.
To help with the number pull requests: