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

Add Clustering Method to PriceTaker #1579

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

MarcusHolly
Copy link
Contributor

Summary/Motivation:

Adds a literature-based method to the pricetaker model for determining the optimal number of clusters that a dataset should be broken into

Changes proposed in this PR:

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@codecov-commenter
Copy link

codecov-commenter commented Feb 25, 2025

Codecov Report

Attention: Patch coverage is 3.22581% with 60 lines in your changes missing coverage. Please review.

Project coverage is 76.80%. Comparing base (5f3b33b) to head (41ee6e0).

Files with missing lines Patch % Lines
...aes/apps/grid_integration/pricetaker/clustering.py 3.50% 55 Missing ⚠️
...s/grid_integration/pricetaker/price_taker_model.py 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1579      +/-   ##
==========================================
- Coverage   76.86%   76.80%   -0.07%     
==========================================
  Files         394      394              
  Lines       63233    63281      +48     
  Branches    10355    10364       +9     
==========================================
- Hits        48604    48602       -2     
- Misses      12184    12235      +51     
+ Partials     2445     2444       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MarcusHolly MarcusHolly marked this pull request as ready for review February 25, 2025 14:05
@ksbeattie ksbeattie added DISPATCHES WaterTAP Priority:Normal Normal Priority Issue or PR labels Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DISPATCHES Priority:Normal Normal Priority Issue or PR WaterTAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants