Skip to content

revise galsim unit tests thresholds #162

Closed

Description

@ismael-mendoza

@beckermr mentioned that some of the thresholds that were changed to allow jax-galsim to pass some GalSim unit tests should be carefully reviewed.

I think all the changes are here: GalSim-developers/GalSim#1252

@beckermr feel free to add more details or if there are any tests in particular you are worried about

Activity

  1. added this to the First Paper milestone on Feb 6, 2026
  2. ismael-mendoza commented on Mar 27, 2026

    @ismael-mendoza
    CollaboratorAuthor
  3. beckermr commented on May 9, 2026

    @beckermr
    Collaborator

    Alright this is done. I fixed a bug in photon shooting for interpolated images and increased the accuracy of the k-space interpolant for the Lanczos kernels. I removed the moffat test suite changes that are no longer executed. So I think we are good to go! Closing!

  4. beckermr commented on May 13, 2026

    @beckermr
    Collaborator

    I finished one more followup item here, addressing @rmjarvis's comments from his review of the same PR. xref: GalSim-developers/GalSim#1360

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    testsRelated to unit-testing

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions