Skip to content

REL: backport #960 to v2.5.2#961

Merged
mj-will merged 2 commits into
bilby-dev:release/2.5.2from
mj-will:backport-#960-to-v2.5.2
Jun 3, 2025
Merged

REL: backport #960 to v2.5.2#961
mj-will merged 2 commits into
bilby-dev:release/2.5.2from
mj-will:backport-#960-to-v2.5.2

Conversation

@mj-will

@mj-will mj-will commented Jun 2, 2025

Copy link
Copy Markdown
Collaborator

Back port #960 to v2.5.x to make v2.5.2.

ColmTalbot and others added 2 commits June 2, 2025 17:04
…v#960)

* BUG: fix the filename extension precedence for result files

* Apply suggestions from code review

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* FMT: add empty line

* TYPO: fix typo in warning

* FMT: remove extra whitespace

* BUG: move default extension fallback

---------

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>
@mj-will mj-will added the release label Jun 2, 2025
@mj-will mj-will added this to the v2.5.2 milestone Jun 2, 2025
@mj-will

mj-will commented Jun 2, 2025

Copy link
Copy Markdown
Collaborator Author

I've checked and I think this would produce the correct version:

$ git log -5 --oneline
66b74c5f (HEAD -> backport-#960-to-v2.5.2, origin/backport-#960-to-v2.5.2) CHANGELOG: prepare v2.5.2
d02b08f7 BUG: fix the filename extension precedence for result files (#960)
83245aee (tag: v2.5.1, upstream/backport/v2.5.2, backport/v2.5.2) CHANGELOG: update for v2.5.1 (#951)
fab9326d BUG: Use "geocent" not "geocenter" where appropriate (#930)
bbf5ac46 BUG: fix injection parameters being fixed during recovery (#931)
$ git tag v2.5.2
$ pip install -e . --dry-run --no-deps
Obtaining file:///home/michael/git_repos/bilby
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... done
Would install bilby-2.5.2

@ColmTalbot ColmTalbot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm happy to merge this, can you do a test run of what version we get on main after this tag is made?

@mj-will

mj-will commented Jun 3, 2025

Copy link
Copy Markdown
Collaborator Author

I'm happy to merge this, can you do a test run of what version we get on main after this tag is made?

Good point, I checked and it does get a bit messy:

$ git log --oneline -n 5
f6b6258d (HEAD -> main, upstream/main, upstream/HEAD) BUG: fix the filename extension precedence for result files (#960)
94e66a21 Merge pull request #958 from mattpitkin/doc_fix
6258f9d9 run_sampler: correct use_ratio docs
22cc23a1 BUG: Fix sampling efficiency warning (#953)
6b3fb3af ENH: plot_multiple: Allow plotting onto user-defined figure (#946)
(nessai) [michael@r9l-icgphd7 bilby]$ python -m setuptools_scm
2.5.2.dev6+gf6b6258d

If the tag were in main, I think it should be 2.5.3.dev#

This won't happen once #954 is merged but in the meantime we'll have to decide how to handle this. I think we have two options:

  • Accept the version will be wrong until we make the next tag
  • Merge the release/v2.5.2 branch back in to main after the release has been made

What do you think?

@ColmTalbot

Copy link
Copy Markdown
Collaborator

This won't happen once #954 is merged but in the meantime we'll have to decide how to handle this. I think we have two options:

  • Accept the version will be wrong until we make the next tag
  • Merge the release/v2.5.2 branch back in to main after the release has been made
    What do you think?

I think the latter here is better, I don't think we should delay the release too much trying to figure out the best way to do this.

@mj-will

mj-will commented Jun 3, 2025

Copy link
Copy Markdown
Collaborator Author

This won't happen once #954 is merged but in the meantime we'll have to decide how to handle this. I think we have two options:

  • Accept the version will be wrong until we make the next tag
  • Merge the release/v2.5.2 branch back in to main after the release has been made
    What do you think?

I think the latter here is better, I don't think we should delay the release too much trying to figure out the best way to do this.

Okay, sounds good to me.

@mj-will mj-will merged commit 33b5cf0 into bilby-dev:release/2.5.2 Jun 3, 2025
mj-will added a commit to mj-will/bilby that referenced this pull request Oct 16, 2025
* BUG: fix the filename extension precedence for result files (bilby-dev#960)

* BUG: fix the filename extension precedence for result files

* Apply suggestions from code review

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* FMT: add empty line

* TYPO: fix typo in warning

* FMT: remove extra whitespace

* BUG: move default extension fallback

---------

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* CHANGELOG: prepare v2.5.2

---------

Co-authored-by: Colm Talbot <talbotcolm@gmail.com>
mj-will added a commit to mj-will/bilby that referenced this pull request Feb 19, 2026
* BUG: fix the filename extension precedence for result files (bilby-dev#960)

* BUG: fix the filename extension precedence for result files

* Apply suggestions from code review

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* FMT: add empty line

* TYPO: fix typo in warning

* FMT: remove extra whitespace

* BUG: move default extension fallback

---------

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* CHANGELOG: prepare v2.5.2

---------

Co-authored-by: Colm Talbot <talbotcolm@gmail.com>
mj-will added a commit to mj-will/bilby that referenced this pull request Feb 19, 2026
* BUG: fix the filename extension precedence for result files (bilby-dev#960)

* BUG: fix the filename extension precedence for result files

* Apply suggestions from code review

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* FMT: add empty line

* TYPO: fix typo in warning

* FMT: remove extra whitespace

* BUG: move default extension fallback

---------

Co-authored-by: Michael J. Williams <michaeljw1@googlemail.com>

* CHANGELOG: prepare v2.5.2

---------

Co-authored-by: Colm Talbot <talbotcolm@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants