Skip to content

[com4]: Add com4 benchmark tests#1321

Open
PaulaSp3 wants to merge 1 commit into
masterfrom
PS_benchmark
Open

[com4]: Add com4 benchmark tests#1321
PaulaSp3 wants to merge 1 commit into
masterfrom
PS_benchmark

Conversation

@PaulaSp3

@PaulaSp3 PaulaSp3 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Added benchmark data:

Added a script that run a simulation and compares the results (pixel wise) with reference data.
Comparison results are written into: avaframe/tests/reportsCom4FlowPy/standardTestsReportCom4FlowPy.md
There are many options to extent these comparisons and what we could write into the report (e.g., plot with the difference of the rasters,..)

resolves #1227

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)

@PaulaSp3
PaulaSp3 requested a review from ahuber-bfw July 22, 2026 14:13
@PaulaSp3 PaulaSp3 self-assigned this Jul 22, 2026
@PaulaSp3 PaulaSp3 added the flowPyDev Ideas for future development label Jul 22, 2026
@qltysh

qltysh Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Qlty


Coverage Impact

This PR will not change total coverage.

🚦 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

flowPyDev Ideas for future development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[com4]: benchmark tests

1 participant