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 new n.add #1316

Merged
merged 2 commits into from
Sep 25, 2024
Merged

prepare for new n.add #1316

merged 2 commits into from
Sep 25, 2024

Conversation

lkstrp
Copy link
Member

@lkstrp lkstrp commented Sep 20, 2024

  • Adds some more explicit indexing inprepare_sector_network.py since non matching indices are not allowed with new PyPSA version
  • See feat: improved n.add PyPSA#896

Copy link
Contributor

github-actions bot commented Sep 20, 2024

Validator Report

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

General

Plots comparison
Main branch Feature branch
Image not available Image not available
Image not available Image not available
Image not available Image not available
Image not available Image not available
Image not available Image not available
Image not available Image not available
Files comparison
Status NRMSE MAE (norm)
nodal_capacities.csv ⚠️ NaN mismatch
nodal_supply_energy.csv ⚠️ NaN mismatch
nodal_costs.csv ⚠️ NaN mismatch
nodal_cfs.csv ⚠️ NaN mismatch
cfs.csv ✅ Almost equal 0.047 0.05
costs.csv ✅ Almost equal 0.001 0.01
price_statistics.csv ✅ Almost equal 0.008 0.14
curtailment.csv ✅ Almost equal 0.000 0.25
metrics.csv ✅ Almost equal 0.000 0.07
supply.csv ✅ Almost equal 0.000 0.02
capacities.csv ✅ Almost equal 0.000 0.03
energy.csv ✅ Almost equal 0.000 0.02
weighted_prices.csv ✅ Almost equal 0.698 0.55
supply_energy.csv ✅ Almost equal 0.000 0.02
prices.csv ✅ Almost equal 0.000 0.15
market_values.csv ✅ Almost equal 0.000 0.08

NRMSE: Normalized (min-max) Root Mean Square Error
MAE (norm): Mean Absolute Error on normalized Data (min-max
Status Threshold: MAE (norm) > 0.05 and NRMSE > 2

Model Metrics

Benchmarks Image not available Image not available Image not available

Comparing madd-preparation (b4ec511) with master (0d1c36b).
Branch is 2 commits ahead and 10 commits behind.
Last updated on 2024-09-24 10:19:42 CEST.

Copy link
Member

@fneum fneum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! So the new n.add would throw a warning if indices or snapshots for time-varying data don't match?

What about switching from n.madd to n.add? That was also planned, right?

scripts/prepare_sector_network.py Outdated Show resolved Hide resolved
Co-authored-by: Fabian Neumann <fabian.neumann@outlook.de>
@lkstrp
Copy link
Member Author

lkstrp commented Sep 24, 2024

Looks good! So the new n.add would throw a warning if indices or snapshots for time-varying data don't match?

Error, not a warning. So the next pypsa release is breaking. Slightly, but breaking.

What about switching from n.madd to n.add? That was also planned, right?

Yes, but this will fail for pypsa<=30.2. So once we set the dependency here on pypsa>=31.0 we can replace all n.madd (and n.madd will just point to n.add with a Deprecation Warning`).

@lkstrp lkstrp merged commit 24e0ddd into master Sep 25, 2024
9 of 10 checks passed
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.

2 participants