oemof / oemof-tabular

Load oemof energy systems from tabular data sources.

https://oemof-tabular.readthedocs.io/

BSD 3-Clause "New" or "Revised" License

7 stars 5 forks source link

Tests for BEV facades #142

Closed SabineHaas closed 12 months ago

SabineHaas commented 1 year ago

Description

Adds tests for the BEV-facade, being developed in #94. For a full list of tests see #139

Fixes #139

Type of change

Please tick or delete options that are not relevant.

[x] Bug fix: move conversion_factors of v2g converter and commodity_converter to output flows
[x] New feature: Tests for BEV-facade (dispatch)
[x] New feature: Tests for BEV-facade (investment)

Checklist:

Please tick or delete options that are not relevant.

[x] New and adjusted code is formatted using the pre-commit hooks
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] If new packages are needed, I added them the setup.py
[x] I have added new features/fixes to the CHANGELOG
[x] I have added my name to AUTHORS

SabineHaas commented 1 year ago

@nailend this PR is ready to review. This PR should be merged before you go on working on PR #94 to avoid merge conflicts. I have opened issue #144, to remind us of the tests that need adaptions.