Skip to content

Updated ftester.py to help with looping behavior.#158

Open
SafferA wants to merge 1 commit intonanograv:mainfrom
SafferA:fTestOverhaul
Open

Updated ftester.py to help with looping behavior.#158
SafferA wants to merge 1 commit intonanograv:mainfrom
SafferA:fTestOverhaul

Conversation

@SafferA
Copy link
Copy Markdown
Collaborator

@SafferA SafferA commented Mar 4, 2026

Slight tweaks to the ftester.py script were made to help with the looping behavior.

  1. There are different thresholds for adding/removing parameters now. Now more strict to remove.
  2. Outputs a config file entry that can be copied into the .yaml file that allows for record keeping of the FTest recommendations.

This is by no means finalized. Formatting for the output will almost certainly change as tests are run and we see what clarifications need to be made.

Comments, critiques, criticisms, and witty banter are welcome.

@SafferA SafferA self-assigned this Mar 4, 2026
@SafferA SafferA requested review from rossjjennings and removed request for rossjjennings March 4, 2026 19:28
@SafferA SafferA marked this pull request as ready for review March 10, 2026 17:23
@SafferA SafferA added enhancement New feature or request discussed Discussed during a NANOGrav GitFlowers Meeting. labels Mar 10, 2026
@SafferA SafferA linked an issue Mar 10, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discussed Discussed during a NANOGrav GitFlowers Meeting. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

F-test overhaul for NG20

1 participant