GalSim-developers / GalSim-developers/GalSim

streamlined, simplified validation scripts

Open
#574 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

cleanup tests
Dominant language
Python
Stars
272
Forks
121
PR merge metrics
No merged PRs in 30d

Description

As discussed in https://github.com/GalSim-developers/GalSim/pull/568#issuecomment-47561275 and subsequent comments, we may want to make a simplified version of the scripts that did the main GalSim validation tests for the GalSim paper, so that we can check that they are still okay when we tag a new version. The latest proposal on that issue was to focus on the 3 key tests that we did, for just the default values of GSParams / interpolants. More discussion and/or work can happen here on this issue.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the discussion in pull request 568 and its subsequent comments to recover the proposed scope. Identify the three key GalSim paper validation tests and how their default GSParams and interpolant settings are currently exercised. Done means a simplified, repeatable set of scripts can check those tests when a new version is tagged.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.