Skip to content

Fix retrieve_eurostat_data for Windows#1825

Merged
fneum merged 4 commits intoPyPSA:masterfrom
Eric-Nitschke:fix_eurostat
Sep 4, 2025
Merged

Fix retrieve_eurostat_data for Windows#1825
fneum merged 4 commits intoPyPSA:masterfrom
Eric-Nitschke:fix_eurostat

Conversation

@Eric-Nitschke
Copy link
Copy Markdown
Contributor

Closes #1818 .

Changes proposed in this Pull Request

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

Eric-Nitschke and others added 4 commits September 3, 2025 15:13
@fneum fneum merged commit 6b2b9c9 into PyPSA:master Sep 4, 2025
8 checks passed
measrainsey added a commit to open-energy-transition/pypsa-wal that referenced this pull request Oct 16, 2025
* Fix retrieve_eurostat_data for Windows (PyPSA#1825)

* Fix retrieve_eurostat_data for Windows

Bugfix:
- close the temporary file in retrieve_eurostat_data and retrieve_eurostat_household_data before accessing it again to unzip it

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update the release_notes

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* fix: adjust for pypsa v1 add return change (PyPSA#1827)

* Update locked environment files for all platforms (PyPSA#1820)

Co-authored-by: lkstrp <[email protected]>
Co-authored-by: Lukas Trippe <[email protected]>

* Oetc support (#50)

* Add oetc configuration support

Co-authored-by: Fabian Hofmann <[email protected]>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Fabian Hofmann <[email protected]>

* Update locked environment files for all platforms (PyPSA#1832)

Co-authored-by: lkstrp <[email protected]>

* remove pinned environment reference from pre-commit-config (PyPSA#1837)

* remove pinned environment reference from pre-commit-config

* release notes

* refactor: use single-line comment for rst (PyPSA#1839)

* Check for leaked secrets using pre-commit (#59)

* Update the other pre-commit tasks

* Revert pre-commit versions

* doc: clean headers (#51)

* doc: clean headers

* doc: improve PR template

* Apply suggestions from code review

Co-authored-by: Johannes HAMPP <[email protected]>

* refactor: adjust header for new page

* fix: adjust header for OETC

---------

Co-authored-by: Johannes HAMPP <[email protected]>

* Fix JRC ARDECO API change. (PyPSA#1850)

---------

Co-authored-by: Eric Nitschke <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lukas Trippe <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: lkstrp <[email protected]>
Co-authored-by: Thomas Gilon <[email protected]>
Co-authored-by: Kristijan Faust <[email protected]>
Co-authored-by: Fabian Hofmann <[email protected]>
Co-authored-by: Fabrizio Finozzi <[email protected]>
Co-authored-by: Will Usher <[email protected]>
Co-authored-by: Johannes HAMPP <[email protected]>
Co-authored-by: Bobby Xiong <[email protected]>
measrainsey added a commit to open-energy-transition/pypsa-wal that referenced this pull request Oct 20, 2025
* Fix retrieve_eurostat_data for Windows (PyPSA#1825)

* Fix retrieve_eurostat_data for Windows

Bugfix:
- close the temporary file in retrieve_eurostat_data and retrieve_eurostat_household_data before accessing it again to unzip it

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update the release_notes

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* fix: adjust for pypsa v1 add return change (PyPSA#1827)

* Update locked environment files for all platforms (PyPSA#1820)

Co-authored-by: lkstrp <[email protected]>
Co-authored-by: Lukas Trippe <[email protected]>

* Oetc support (#50)

* Add oetc configuration support

Co-authored-by: Fabian Hofmann <[email protected]>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Fabian Hofmann <[email protected]>

* Update locked environment files for all platforms (PyPSA#1832)

Co-authored-by: lkstrp <[email protected]>

* remove pinned environment reference from pre-commit-config (PyPSA#1837)

* remove pinned environment reference from pre-commit-config

* release notes

* refactor: use single-line comment for rst (PyPSA#1839)

* Check for leaked secrets using pre-commit (#59)

* Update the other pre-commit tasks

* Revert pre-commit versions

* doc: clean headers (#51)

* doc: clean headers

* doc: improve PR template

* Apply suggestions from code review

Co-authored-by: Johannes HAMPP <[email protected]>

* refactor: adjust header for new page

* fix: adjust header for OETC

---------

Co-authored-by: Johannes HAMPP <[email protected]>

* Fix JRC ARDECO API change. (PyPSA#1850)

---------

Co-authored-by: Eric Nitschke <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lukas Trippe <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: lkstrp <[email protected]>
Co-authored-by: Thomas Gilon <[email protected]>
Co-authored-by: Kristijan Faust <[email protected]>
Co-authored-by: Fabian Hofmann <[email protected]>
Co-authored-by: Fabrizio Finozzi <[email protected]>
Co-authored-by: Will Usher <[email protected]>
Co-authored-by: Johannes HAMPP <[email protected]>
Co-authored-by: Bobby Xiong <[email protected]>
toniseibold pushed a commit that referenced this pull request Dec 1, 2025
* Fix retrieve_eurostat_data for Windows

Bugfix:
- close the temporary file in retrieve_eurostat_data and retrieve_eurostat_household_data before accessing it again to unzip it

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update the release_notes

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

retrieve_eurostat_household_data permission error on Windows

2 participants