Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Oct 6, 2025

Automated via .github/workflows/weekly_update.yml.

@github-actions github-actions bot added the component: third party Changes in WarpX that reflect a change in a third-party library label Oct 6, 2025
@EZoni EZoni self-requested a review October 6, 2025 14:20
@EZoni EZoni self-assigned this Oct 6, 2025
@EZoni
Copy link
Member

EZoni commented Oct 6, 2025

I think this needs AMReX-Codes/pyamrex#484 first.

@EZoni
Copy link
Member

EZoni commented Oct 7, 2025

@RemiLehe @ax3l

Here we got the checksums updates that @WeiqunZhang had warned about, supposedly related to AMReX-Codes/amrex#4658 and its relatives (AMReX-Codes/amrex#4700 and AMReX-Codes/pyamrex#484).

If I find a moment I would like to open a separate PR where I add a feature to output also the maximum relative error for each checksum test that fails, at the end of the output we already have, so it's easier to have an estimate of that.

@EZoni
Copy link
Member

EZoni commented Oct 7, 2025

I suspect the PETSc workflows need a separate fix. Never mind, I think it's just a temporary server-side issue.

@EZoni
Copy link
Member

EZoni commented Oct 7, 2025

If I find a moment I would like to open a separate PR where I add a feature to output also the maximum relative error for each checksum test that fails, at the end of the output we already have, so it's easier to have an estimate of that.

I implemented this small change in #6252.

RemiLehe pushed a commit that referenced this pull request Oct 7, 2025
This PR adds a print of the maximum relative error when checksums fail,
which I think is quite useful when they fail for many "keys" and/or many
tests (e.g., #5955, #6248) - the maximum relative error gives a glimpse
of how good or bad things are.

This is an example of the new output, with the extra line printing the
maximum relative error:
```
...

ERROR: Benchmark and output file checksum have different value for key [lev=0,Ex]
Benchmark: [lev=0,Ex] 9.408402900367834e+05
Test file: [lev=0,Ex] 9.409402900367834e+05
Absolute error: 1.00e+02
Relative error: 1.06e-04
ERROR: Benchmark and output file checksum have different value for key [lev=0,Ez]
Benchmark: [lev=0,Ez] 6.654423300420902e+05
Test file: [lev=0,Ez] 6.754423300420902e+05
Absolute error: 1.00e+04
Relative error: 1.50e-02
ERROR: Benchmark and output file checksum have different value for key [lev=0,jx]
Benchmark: [lev=0,jx] 9.956269465957083e+03
Test file: [lev=0,jx] 9.956269365957083e+03
Absolute error: 1.00e-04
Relative error: 1.00e-08

Maximum relative error: 1.50e-02

New checksums file test_2d_larmor.json:

...
```

Feel free to close the PR without merging if you think this is not
useful.
@EZoni EZoni merged commit ab947bd into development Oct 7, 2025
51 of 53 checks passed
@EZoni EZoni deleted the weekly_update branch October 7, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: third party Changes in WarpX that reflect a change in a third-party library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants