Skip to content

fix: use slower maxk comp to match galsim#229

Merged
beckermr merged 2 commits intomainfrom
update-tests-1
May 4, 2026
Merged

fix: use slower maxk comp to match galsim#229
beckermr merged 2 commits intomainfrom
update-tests-1

Conversation

@beckermr
Copy link
Copy Markdown
Collaborator

@beckermr beckermr commented May 4, 2026

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 4, 2026

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

✅ 36 untouched benchmarks


Comparing update-tests-1 (084b60d) with main (d91fcda)

Open in CodSpeed

@beckermr beckermr changed the title test: run tests against latest merge of main into fix-tests fix: use slower maxk comp to match galsim May 4, 2026
@beckermr
Copy link
Copy Markdown
Collaborator Author

beckermr commented May 4, 2026

@ismael-mendoza with the galsim maxk bug fixed, we now need to more complicated maxk comp to pass the test suite. This PR updates the test suite and switches to that.

@beckermr beckermr merged commit 7b3c107 into main May 4, 2026
10 checks passed
@beckermr beckermr deleted the update-tests-1 branch May 4, 2026 19:00
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.

1 participant