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

Feature #70/multi charging events #108

Merged
merged 6 commits into from
Nov 29, 2023
Merged

Conversation

mosc5
Copy link
Member

@mosc5 mosc5 commented Oct 29, 2023

Fixes #70

This PR should be merged after #105

Changes proposed in this pull request:

  • Avoid too many vehicles charging at once. Implement basic case, if at any point during the planned charging event the number of possbile chargees is exceeded at a Location, the charging event will be denied

The following steps were realized (required):

  • Correct linting and formatting with flake8 src & mypy src & black src
  • Check if tests pass locally with pytest
  • Run tox
  • Add the description of your changes to the CHANGELOG.md in subsection with release name v0.0.0-alpha - 2022-04-28 and state the PR number with #

Also the following steps were realized if applicable:

  • Write docstrings to your code
  • Write test(s) for new patch of code

@mosc5 mosc5 requested a review from alexkens October 29, 2023 13:05
@mosc5 mosc5 force-pushed the feature-#70/multi_charging_events branch from 279753f to 5bd4fce Compare November 29, 2023 15:01
@mosc5 mosc5 merged commit 7b40e57 into dev Nov 29, 2023
4 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.

Implement charging of multiple vehicles at the same time
2 participants