Skip to content

New tx cost inputs - #241

Open
ppinchuk wants to merge 124 commits into
mainfrom
pp/new_tx_cost_inputs
Open

ppinchuk wants to merge 124 commits into
mainfrom
pp/new_tx_cost_inputs

Conversation

@ppinchuk

@ppinchuk ppinchuk commented Feb 11, 2025 •

Copy link
Copy Markdown
Collaborator

Add more flexible method to specify costs for LCP routing. Users can now combine costs and scale them with just the config input, allowing for more flexible scenario creation.

@ppinchuk ppinchuk self-assigned this Feb 11, 2025
@ppinchuk ppinchuk added refactor Code refactor or update topic-transmission Issues/pull requests related to LCP/transmission routing labels Feb 11, 2025
@codecov-commenter

codecov-commenter commented Feb 11, 2025 •

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 75.40323% with 183 lines in your changes missing coverage. Please review.

Project coverage is 74.02%. Comparing base (48af1b2) to head (40fec7e).

Files with missing lines Patch % Lines
...eVX/least_cost_xmission/least_cost_xmission_cli.py 27.58% 84 Missing ⚠️
reVX/least_cost_xmission/least_cost_paths.py 88.25% 21 Missing and 8 partials ⚠️
reVX/least_cost_xmission/least_cost_paths_cli.py 50.84% 25 Missing and 4 partials ⚠️
reVX/least_cost_xmission/least_cost_xmission.py 80.67% 17 Missing and 6 partials ⚠️
reVX/least_cost_xmission/trans_cap_costs.py 90.64% 10 Missing and 6 partials ⚠️
reVX/config/least_cost_xmission.py 92.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   74.59%   74.02%   -0.58%     
==========================================
  Files          75       75              
  Lines       10367    10528     +161     
  Branches     1204     1232      +28     
==========================================
+ Hits         7733     7793      +60     
- Misses       2143     2237      +94     
- Partials      491      498       +7     
Flag Coverage Δ
unittests 74.02% <75.40%> (-0.58%) ⬇️

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

Files with missing lines Coverage Δ
reVX/config/transmission_layer_creation.py 100.00% <100.00%> (ø)
reVX/least_cost_xmission/config/xmission_config.py 65.30% <100.00%> (-21.80%) ⬇️
reVX/config/least_cost_xmission.py 86.55% <92.00%> (-0.43%) ⬇️
reVX/least_cost_xmission/trans_cap_costs.py 87.31% <90.64%> (+5.08%) ⬆️
reVX/least_cost_xmission/least_cost_xmission.py 79.71% <80.67%> (-3.68%) ⬇️
reVX/least_cost_xmission/least_cost_paths.py 88.88% <88.25%> (-6.20%) ⬇️
reVX/least_cost_xmission/least_cost_paths_cli.py 56.52% <50.84%> (-4.39%) ⬇️
...eVX/least_cost_xmission/least_cost_xmission_cli.py 44.85% <27.58%> (-4.13%) ⬇️

... and 1 file with indirect coverage changes

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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code refactor or update topic-transmission Issues/pull requests related to LCP/transmission routing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants