Skip to content

Bring dev branch into parity with "recent" commits to main#128

Merged
kayhangultekin merged 5 commits intodevfrom
main
Nov 25, 2025
Merged

Bring dev branch into parity with "recent" commits to main#128
kayhangultekin merged 5 commits intodevfrom
main

Conversation

@kayhangultekin
Copy link
Copy Markdown
Collaborator

@kayhangultekin kayhangultekin commented Nov 21, 2025

Description

dev (main development branch) is behind main in a number of bug-fix commits. I think these are all straightforward non-breaking should-be-implemented bug fixes. Please review and give a thumbs-up if you agree.

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Will bring dev up in line with main

Questions

Status

Shreyas Tiruvaskar and others added 5 commits February 26, 2025 14:19
…urce strains in gravwaves._gws_harmonics_at_evo_fobs()
minor bugfix to add missing factor of 1/dlnf in definition of loud so…
Fixing errors in param_spaces_classic.py

I believe the code in question was created to create a user-friendly way of reproducing the NANOGrav 15yr Astro Interpretation paper. It was not the code that we actually used for that analysis. This fix corrects a mistake introduced in an attempt to do "open science".
* Fixing errors in param_spaces_classic.py

* Fix bug in sams/components.py

---------

Co-authored-by: Shreyas Tiruvaskar <sti50@bohr.canterbury.ac.nz>
@kayhangultekin kayhangultekin self-assigned this Nov 21, 2025
@kayhangultekin kayhangultekin added the bug Something isn't working label Nov 21, 2025
Copy link
Copy Markdown
Collaborator

@CayenneMatt CayenneMatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that these changes should be added into dev.

Copy link
Copy Markdown
Collaborator Author

@kayhangultekin kayhangultekin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, and I got confirmation from @CayenneMatt. I am approving for merge.

@kayhangultekin kayhangultekin merged commit 6473d87 into dev Nov 25, 2025
12 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants