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

Make lol and temperatures optional. make cfg parameter backwards compatible #211

Merged
merged 6 commits into from
Sep 24, 2024

Conversation

PaulScheerRLI
Copy link
Collaborator

Simulation can run without temperatures and level of loadings if vehicle_types have constant mileage.
If vehicle types with mileage look need one of these values during simulation, the simulation will fail. Test added to check this feature.

Deprecated arguments:

  • electrified_stations
  • vehicle_types
  • cost_parameters_file

can be used again, but will throw a warning to use the updated naming.
Station Height data lookup will give errors in case station is missing from station_data. In other cases a single warning is given if no station data is provided.

…er names backwards compatible

Simulation can run without temperatures and lol if vehicle_types have constant mileage. vehicle types with mileage look up will fail late. Arguments expanded with deprecated values which replace current values. Warning for deprecation is given. Station Height data lookup will give errors in case station is missing from station_data. in other cases a single warning is given if no station data is provided
Copy link
Collaborator

@j-brendel j-brendel left a comment

Choose a reason for hiding this comment

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

Nice

@j-brendel j-brendel merged commit 7b4bbda into dev Sep 24, 2024
6 checks passed
@j-brendel j-brendel deleted the bugfix/optional_timeseries_and_compatibility branch September 24, 2024 11:03
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