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 pipe roughness to API's get_simulation_array #368

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

ThalesCarl
Copy link
Contributor

Added to be used by corrosion plugin's mixture model calculation.

The nomenclature ks was chosen instead of ε because it's already being used by get_ucm_friction_factor_input_variable hook

DCC-182

Copy link

codecov bot commented Mar 12, 2024

Codecov Report

Merging #368 (8db621b) into master (0e5ef12) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #368   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files          55       55           
  Lines        6966     6966           
=======================================
  Hits         6964     6964           
  Misses          2        2           

Added to be used by corrosion plugin's mixture model calculation.

DCC-182
@ThalesCarl ThalesCarl force-pushed the fb-DCC-182-mixture-model branch from 14d0875 to 8db621b Compare March 27, 2024 12:21
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ThalesCarl ThalesCarl merged commit 13ae701 into master Mar 27, 2024
18 checks passed
@ThalesCarl ThalesCarl deleted the fb-DCC-182-mixture-model branch March 27, 2024 14:21
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