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

Household hp ac ts ev #405

Draft
wants to merge 140 commits into
base: main
Choose a base branch
from
Draft

Household hp ac ts ev #405

wants to merge 140 commits into from

Conversation

Manish-Khanra
Copy link
Contributor

Pull Request

Related Issue

Closes #[issue number] (if applicable)

Description

[Provide a brief description of the changes in this PR]

Changes Proposed

  • [List the main changes or features implemented]

Testing

[Describe the testing you've done, including any specific test cases or scenarios]

Checklist

Please check all applicable items:

  • Code changes are sufficiently documented (docstrings, inline comments, doc folder updates)
  • New unit tests added for new features or bug fixes
  • Existing tests pass with the changes
  • Reinforcement learning examples are operational (for DRL-related changes)
  • Code tested with both local and Docker databases
  • Code follows project style guidelines and best practices
  • Changes are backwards compatible, or deprecation notices added
  • New dependencies added to pyproject.toml
  • A note for the release notes doc/release_notes.rst of the upcoming release is included
  • Consent to release this PR's code under the GNU Affero General Public License v3.0

Additional Notes (if applicable)

[Any additional information, concerns, or areas you want reviewers to focus on]

Screenshots (if applicable)

[Add screenshots to demonstrate visual changes]

Manish-Khanra and others added 30 commits August 2, 2023 09:41
-fixed components
-adjusted the bidding strategy
@Manish-Khanra Manish-Khanra marked this pull request as draft August 19, 2024 07:44
@Manish-Khanra Manish-Khanra self-assigned this Aug 19, 2024
assume/units/building.py Outdated Show resolved Hide resolved
assume/units/building.py Outdated Show resolved Hide resolved
assume/units/building.py Show resolved Hide resolved
assume/strategies/naive_strategies.py Outdated Show resolved Hide resolved
assume/units/building.py Outdated Show resolved Hide resolved
)

self.electricity_price = self.forecaster["price_EOM"]
self.natural_gas_price = self.forecaster["fuel_price_natural_gas"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

fuel_price_natural_gas needs to be called fuel_price_natural gas since in the input file it is written natural gas too. Or the csv_loader gets adapted such that it replaces whitespaces with "_".

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.

4 participants