Skip to content

Bug fix/ev usage power limit - #114

Merged
explodingdinosaurs merged 4 commits into
mainfrom
bug_fix/ev_usage_power_limit
Jul 28, 2026
Merged

Bug fix/ev usage power limit#114
explodingdinosaurs merged 4 commits into
mainfrom
bug_fix/ev_usage_power_limit

Conversation

@explodingdinosaurs

Copy link
Copy Markdown
Contributor

This PR:

  • Fixes a bug in EVBase._create_vehicle_port() wherein the creation of the MobileElectricalStorage should receive discharging_power_limit=self.usage_power_limit not discharging_power_limit=self.discharging_power_limit.
  • Adds tests for the above change.
  • Bumps version to 2.2.3

@mikejturner

Copy link
Copy Markdown
Contributor

✅ Code coverage on bug_fix/ev_usage_power_limit is at 81% (compared to main being at 81%)

🎉 No files have reduced coverage 🎉

Full Coverage Report

File Details
src/echo/__init__.py ✅ 100% → 100%
src/echo/configuration.py ✅ 100% → 100%
src/echo/constants.py ✅ 100% → 100%
src/echo/exceptions.py ✅ 88% → 88%
src/echo/functional.py ✅ 100% → 100%
src/echo/models/__init__.py ✅ 100% → 100%
src/echo/models/agnostic.py ✅ 86% → 86%
src/echo/models/base.py ✅ 83% → 83%
src/echo/models/carbon.py ✅ 100% → 100%
src/echo/models/electrical.py ✅ 74% → 74%
src/echo/models/gas.py ✅ 73% → 73%
src/echo/models/prebuilt.py ✅ 56% → 56%
src/echo/models/scenario.py ✅ 100% → 100%
src/echo/models/thermal.py ✅ 81% → 81%
src/echo/objectives/__init__.py ✅ 100% → 100%
src/echo/objectives/base.py ✅ 98% → 98%
src/echo/objectives/contingency.py ✅ 93% → 93%
src/echo/objectives/power.py ✅ 86% → 86%
src/echo/objectives/tariff.py ✅ 80% → 80%
src/echo/optimiser.py ✅ 78% → 78%
src/echo/tracker.py ✅ 43% → 43%
src/echo/utils.py ✅ 61% → 61%
src/echo/validators.py ✅ 89% → 89%
src/echo/visualization.py ✅ 91% → 91%
TOTAL ✅ 81% → 81%

@explodingdinosaurs
explodingdinosaurs merged commit d7d5ec1 into main Jul 28, 2026
11 checks passed
@explodingdinosaurs
explodingdinosaurs deleted the bug_fix/ev_usage_power_limit branch July 28, 2026 06:05
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