Skip to content

[com1]: Update report com1#1315

Open
awirb wants to merge 2 commits into
masterfrom
updateReportCom1
Open

[com1]: Update report com1#1315
awirb wants to merge 2 commits into
masterfrom
updateReportCom1

Conversation

@awirb

@awirb awirb commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

add info in com1DFA report that projected area and actual area are derived from the release raster and hence are an approximation of the polygon area (deviations are a function of mesh cell size)

PR Checklist

Please confirm before requesting review:

  • I ran pytest locally without fails
  • I added/updated tests where needed
  • I updated documentation where needed

Confirm before the final merge/rebase into master

  • Commits are sensibly squashed and rebased onto latest master
  • Standardtest run without difference (with recompiled cython code)

@awirb awirb requested review from JuLa96 and fso42 July 8, 2026 13:49
@awirb awirb self-assigned this Jul 8, 2026
@awirb awirb added the featureRequest ideas or functions that would be useful label Jul 8, 2026
@awirb awirb changed the title Update report com1 [com1]: Update report com1 Jul 8, 2026
@qltysh

qltysh Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Qlty


Coverage Impact

This PR will not change total coverage.

Modified Components (1)

RatingComponent% Diff
Coverage rating: C Coverage rating: C
com1DFA
🚦 See full report on Qlty Cloud »

🛟 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.

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

Labels

featureRequest ideas or functions that would be useful

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant