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.

Sign up for GitHub

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

Jump to bottom

Battery electric vehicle #94

Draft
wants to merge 98 commits into
base: dev
Choose a base branch
from
Draft

Battery electric vehicle #94

wants to merge 98 commits into from

Conversation

jnnr
Copy link
Member

@jnnr jnnr commented Feb 14, 2023 •
edited by nailend
Loading

close #110

Moved from github.com/rl-institut/oemoflex.

  • Has to be adapted to the new dataclasses facade.
  • Add a tests
  • Validate test
  • validate lp-file
  • Add lp-file

@jnnr jnnr changed the base branch from dev to revision/facades-module February 14, 2023 11:47
Base automatically changed from revision/facades-module to dev February 15, 2023 16:26
@srhbrnds srhbrnds self-assigned this Feb 15, 2023
@nailend nailend mentioned this pull request Jul 13, 2023
Open
@nailend nailend self-assigned this Aug 4, 2023
@nailend nailend added the enhancement New feature or request label Aug 4, 2023
@nailend
Fix label
8a2694c
@nailend
Draft for joint extension constraint
75f8be0
@nailend
Merge branch 'dev' into feature/battery-electric-vehicle
829895d
@nailend
Merge branch 'dev' into feature/battery-electric-vehicle
6f8a4d3
@nailend
Update converter
82707e6
@nailend
Update buses
616ae77
@nailend
Comment sequencing
89061db
@nailend
Rename labels
be8c743
@nailend
Add mobility sector facade
6383a86
@nailend
Merge branch 'dev' into feature/battery-electric-vehicle
31a8a69
@nailend
Merge branch 'dev' into feature/battery-electric-vehicle
944679c
@nailend
Add bev flag to _investment()
a7ff4ad 
If bev is True, all investment parameters are given
but no ep_costs. This is because multiple components
are linked within one bev facade. But only the in flow
of the storage shall have the aggregated costs for the
whole facade.
@nailend
Make bev expandable
fef8d93
@nailend
Add test script for mobility with bev
69754d0
@nailend
Add graph plotting helpers
1cbafcc
@nailend
Add bev to typemap
863b69c
@nailend
Rework bev component
2d276e5
SabineHaas and others added 28 commits November 21, 2023 15:02
@SabineHaas
Rename test class for bev dispatch tests to TestBevFacadesDispatch
5bdc4d3
@SabineHaas
Add solving of test case and first assertion
f20244d
@SabineHaas
Add lines for constraints
c22fc19
@SabineHaas
Corrections
246ee4b
@nailend
Manually add bev constraints
5380dc0
@SabineHaas
Working V2G BEV test with one time step, todos added
fe2c39c
@SabineHaas
Join classes of dispatch and investment tests
fbca239
@SabineHaas
Add BEV investment tests with constraints
0276011
@SabineHaas
Remove unused lines
2d356ad
@SabineHaas
Add to changelog and authors
c04c3f3
@SabineHaas
Minor adaptions to docstrings
4d767e6
@SabineHaas
Remove initial_storage_capacity from tests, and use initial_storage_l…
ba9e455 
…evel
@SabineHaas
Add BEV multi-period investment tests (skip tests)
4c8899e
@SabineHaas
Add BEV multi-period dispatch test for V2G
f1367f8
@SabineHaas
Name issue in reason of skipped tests
d83b685
@nailend
Remove equal-invest constraint from bev constraint lp test
7472a15 
The equal-invest constraint is not necessary anymore as
the flow through the bev-facade is already restricted by the
storage in & outflow. Costs are only applied for the storage.
@nailend
Revert "Remove equal-invest constraint from bev constraint lp test"
7a9d9da 
Although not necessary, we constraint will still be applied to check
the lp files.
This reverts commit 7472a15.
@nailend
Add timesteps to docstring
9eb57a9
@nailend
Add lines to table in docstrings
0c37c8a
@nailend
Add explanation on invest constraint
7410421
@nailend
Adapt bev constraint test
83f3ab7
@nailend
Remove bev multi-period constraint test
a64ef1e 
This test is not necessary as there are no further features
compared to the single period one.
@nailend
Add bev trio constraint lp file
42223c2
@nailend
Merge branch 'dev' into feature/bev_tests
c7aadd1
@nailend
Rename lp file
9774aea
@nailend
Update lp file
3ea89f9
@nailend
Merge pull request #142 from oemof/feature/bev_tests
3d30c87 
Tests for BEV facades
@nailend
Merge branch 'dev' into feature/battery-electric-vehicle
77d4a06
@nailend nailend mentioned this pull request Mar 1, 2024
Open
@SabineHaas SabineHaas mentioned this pull request Mar 28, 2024
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henhuy henhuy henhuy requested changes

@nailend nailend nailend left review comments

Assignees

@srhbrnds srhbrnds

@nailend nailend

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement EV-component
5 participants
@jnnr @SabineHaas @henhuy @nailend @srhbrnds