Skip to content

Allow HSDS resoruce files as input to TechMapping#547

Merged
ppinchuk merged 13 commits intomainfrom
pp/hsds_tm
May 20, 2025
Merged

Allow HSDS resoruce files as input to TechMapping#547
ppinchuk merged 13 commits intomainfrom
pp/hsds_tm

Conversation

@ppinchuk
Copy link
Copy Markdown
Collaborator

When performing techmapping operations and base aggregation methods, we now check for an HSDS resource file and pass the correct keyword if necessary.

@ppinchuk ppinchuk added this to the Repo updates milestone May 19, 2025
@ppinchuk ppinchuk self-assigned this May 19, 2025
@ppinchuk ppinchuk added bugfix Fixes a bug enhancement Update to logic or general code improvements p-high Priority: high labels May 19, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 76.19048% with 10 lines in your changes missing coverage. Please review.

Project coverage is 79.59%. Comparing base (ad32979) to head (152ec7f).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
reV/supply_curve/points.py 14.28% 6 Missing ⚠️
reV/supply_curve/aggregation.py 70.00% 2 Missing and 1 partial ⚠️
reV/config/project_points.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #547      +/-   ##
==========================================
- Coverage   79.60%   79.59%   -0.02%     
==========================================
  Files          63       63              
  Lines       11152    11160       +8     
  Branches     1607     1606       -1     
==========================================
+ Hits         8878     8883       +5     
- Misses       1783     1788       +5     
+ Partials      491      489       -2     
Flag Coverage Δ
unittests 79.59% <76.19%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ppinchuk ppinchuk merged commit 7f59089 into main May 20, 2025
13 checks passed
@ppinchuk ppinchuk deleted the pp/hsds_tm branch May 20, 2025 00:31
github-actions bot pushed a commit that referenced this pull request May 20, 2025
Allow HSDS resoruce files as input to TechMapping
@ppinchuk ppinchuk mentioned this pull request May 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a bug enhancement Update to logic or general code improvements p-high Priority: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants