Skip to content

Conversation

@PaulaSp3
Copy link
Contributor

e.g., the field surface change can have negative values, so for the report plots we need to take an extent with all values (that are positive and negative).
Since all cells that are not hit by the process are 0, we could define the extent that is shown in the plot with values != 0.

@PaulaSp3 PaulaSp3 requested review from awirb and fso42 November 18, 2025 13:14
@PaulaSp3 PaulaSp3 self-assigned this Nov 18, 2025
@fso42 fso42 added the enhancement New feature or request label Nov 18, 2025
@qltysh
Copy link
Contributor

qltysh bot commented Nov 18, 2025

Diff Coverage: The code coverage on the diff in this pull request is 100.0%.

Total Coverage: This PR will not change total coverage.

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@PaulaSp3
Copy link
Contributor Author

standard tests ident

@PaulaSp3 PaulaSp3 merged commit 09e85f4 into master Nov 19, 2025
4 checks passed
@PaulaSp3 PaulaSp3 deleted the PS_reportPlots branch November 19, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants