Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

prepare for pypsa 0.31.0 #1340

Merged
merged 4 commits into from
Oct 15, 2024
Merged

prepare for pypsa 0.31.0 #1340

merged 4 commits into from
Oct 15, 2024

Conversation

lkstrp
Copy link
Member

@lkstrp lkstrp commented Oct 1, 2024

  • Pins pypsa>=0.31.0
  • Replaces all n.madd with n.add. This wouldn't break otherwise, but spams deprecation warnings

Copy link
Contributor

github-actions bot commented Oct 1, 2024

Validator Report

I am the Validator. Download all artifacts here.
I'll be back and edit this comment for each new commit.

❗ Run failed!

Download 'logs' artifact to see more details.

  • master failed in: cluster_network, cluster_network, cluster_network
  • remove-madd passed! ✅

Model Metrics

Benchmarks Image not available Image not available Image not available

Comparing remove-madd (6878ed4) with master (4435f02).
Branch is 4 commits ahead and 0 commits behind.
Last updated on 2024-10-15 12:18:46 CEST.

@davide-f davide-f mentioned this pull request Oct 2, 2024
7 tasks
* test: log env diff (#1338)

* Update mock_snakemake

---------

Co-authored-by: Lukas Trippe <[email protected]>
Co-authored-by: Fabian Neumann <[email protected]>
@lkstrp
Copy link
Member Author

lkstrp commented Oct 14, 2024

Good to go?

@fneum
Copy link
Member

fneum commented Oct 14, 2024

The changes made so far look good, but perhaps other similar deprecations should be addressed in one go?:

/home/runner/work/pypsa-eur/pypsa-eur/.snakemake/scripts/tmpt6904tfn.base_network.py:794: DeprecatedWarning: import_components_from_dataframe is deprecated as of 0.29 and will be removed in 1.0. Use `n.add` instead. E.g. `n.add(class_name, df.index, **df)`.
WARNING:pypsa.components:Single value sequence for type is treated as a scalar and broadcasted to all components. It is recommended to explicitly pass a scalar instead.
/home/runner/work/pypsa-eur/pypsa-eur/.snakemake/scripts/tmp_f84nzgk.simplify_network.py:128: DeprecatedWarning: mremove is deprecated as of 0.31 and will be removed in 1.0. Use `n.remove` as a drop-in replacement instead.
/home/runner/work/pypsa-eur/pypsa-eur/.snakemake/scripts/tmpdvo4xrrd.cluster_network.py:418: DeprecatedWarning: network is deprecated as of 0.32 and will be removed in 1.0. Use `clustering.n` instead.

@lkstrp
Copy link
Member Author

lkstrp commented Oct 15, 2024

Fair

@lkstrp lkstrp merged commit 0e0191a into master Oct 15, 2024
12 of 13 checks passed
@fneum
Copy link
Member

fneum commented Oct 15, 2024

Thanks @lkstrp!

lindnemi added a commit that referenced this pull request Oct 17, 2024
* feat: add temperature reduction to workflow

* chore: fix name change issues

* feat: update input/output of dependent rules

* feat: add {planning _horizons} wildcard to COP input in myopic rules

* fix: exponential temperature reduction

* feat: add {planning_horizons} wildcard to solve_perfect

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

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

* style: "_today" -> "_baseyear"

* doc: update configtables

* style: "_today" -> "_baseyear" (config)

* feat: update heat pump efficiency in add_brownfield

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

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

* fix: pass planning_horizons to add_brownfield

* feat: update COPs for perfect foresight

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

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

* update release notes

* Update doc/release_notes.rst

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

* Update scripts/build_central_heating_temperature_profiles/run.py

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

* Update scripts/build_central_heating_temperature_profiles/run.py

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

* Update build_central_heating_temperature_profiles.run

* style: simplify update_heat_pump_efficiency functions

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

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

* Update README.md

* [pre-commit.ci] pre-commit autoupdate (#1352)

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/psf/black-pre-commit-mirror: 24.8.0 → 24.10.0](psf/black-pre-commit-mirror@24.8.0...24.10.0)

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

* transmission_projects: update AC line types to high-temperature lines (#1348)

* Bugfix of solar constraint in post discretization (#1343)

* Update NEP DC20 to underground

* fix: remove broken pre-commit rule (docformatter) (#1362)

* prepare for pypsa `0.31.0` (#1340)

* prepare for pypsa `0.31.0`

* Update mock_snakemake (#1345)

* test: log env diff (#1338)

* Update mock_snakemake

---------

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

* resolve other deprecations

---------

Co-authored-by: Davide Fioriti <[email protected]>
Co-authored-by: Fabian Neumann <[email protected]>

* [github-actions.ci] Update fixed environment (#1363)

* [create-pull-request] automated change

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

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

---------

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

* fix: more robust `progress_retrieve` (#1361)

* madd to add

---------

Co-authored-by: Amos Schledorn <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Amos Schledorn <[email protected]>
Co-authored-by: Fabian Neumann <[email protected]>
Co-authored-by: Philipp Glaum <[email protected]>
Co-authored-by: Toni Seibold <[email protected]>
Co-authored-by: Tom Brown <[email protected]>
Co-authored-by: Lukas Trippe <[email protected]>
Co-authored-by: Davide Fioriti <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: lkstrp <[email protected]>
@lkstrp lkstrp deleted the remove-madd branch November 25, 2024 10:46
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.

3 participants