Skip to content

Add Rimsky orchestration bridge for RIFT - #185

Open
oshaughnessy-junior wants to merge 11 commits into
oshaughn:rift_O4cfrom
oshaughnessy-junior:rift_O4c_rimsky_integration
Open

oshaughnessy-junior wants to merge 11 commits into
oshaughn:rift_O4cfrom
oshaughnessy-junior:rift_O4c_rimsky_integration

Conversation

@oshaughnessy-junior

@oshaughnessy-junior oshaughnessy-junior commented Sep 2, 2026

Copy link
Copy Markdown

Summary

  • generate a RIFT follow-up analysis for Rimsky sample_sink orchestration
  • normalize Rimsky/Bilby prior and PSD metadata into the RIFT Asimov schema
  • create Rimsky-scoped LAL cache files for local frames and bootstrap RIFT from its PESummary output
  • document the operator workflow and install rift-rimsky-analysis

The branch starts directly from oshaughn:rift_O4c at cac3b3e.

Validation

  • 28 focused Rimsky/bootstrap tests pass with the O4c Asimov 0.5 environment
  • 13 Rimsky integration tests pass with Asimov 0.6.1, the version locked by current Rimsky
  • setup.py metadata resolves the RIFT package successfully
  • git diff --check passes; the branch-wide diff contains no generated run products

@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 2, 2026 16:17 — with GitHub Actions Active
@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 2, 2026 16:30 — with GitHub Actions Active
@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 2, 2026 16:41 — with GitHub Actions Active

oshaughn commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Updated for the operational target requested here:

  • the generated Rimsky config defaults Bilby to full-submit and automatically wires the post-PE RIFT follow-up;
  • the RIFT production is Ready and bootstraps from the first bilby-online PESummary metafile;
  • a real Rimsky → Asimov → RIFT end-to-end test now requires Asimov >=0.7;
  • CI covers Rimsky 0.1.0rc1 on Python 3.12 and pinned Rimsky main on Python 3.14, forcing Asimov 0.7 and the merged bilby_pipe Asimov-0.7 adapter (be6c770).

Local verification passed for both published and current Rimsky combinations. External HTCondor submission is the only boundary excluded from the E2E test.

1 similar comment
@oshaughnessy-junior

Copy link
Copy Markdown
Author

Updated for the operational target requested here:

  • the generated Rimsky config defaults Bilby to full-submit and automatically wires the post-PE RIFT follow-up;
  • the RIFT production is Ready and bootstraps from the first bilby-online PESummary metafile;
  • a real Rimsky → Asimov → RIFT end-to-end test now requires Asimov >=0.7;
  • CI covers Rimsky 0.1.0rc1 on Python 3.12 and pinned Rimsky main on Python 3.14, forcing Asimov 0.7 and the merged bilby_pipe Asimov-0.7 adapter (be6c770).

Local verification passed for both published and current Rimsky combinations. External HTCondor submission is the only boundary excluded from the E2E test.

@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 2, 2026 16:43 — with GitHub Actions Active
@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 2, 2026 16:47 — with GitHub Actions Active

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Independent automated review completed at the recorded exact commit. Detailed findings were withheld from public output by the private-context egress policy and require private human declassification.

@oshaughnessy-junior
oshaughnessy-junior deployed to private-review-dispatch-rift-upstream September 4, 2026 11:35 — with GitHub Actions Active

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Independent automated review completed at the recorded exact commit. Detailed findings were withheld from public output by the private-context egress policy and require private human declassification.

oshaughn pushed a commit that referenced this pull request Sep 4, 2026
oshaughn added a commit that referenced this pull request Sep 4, 2026
…based

CIP: --fit-distance-tail, so the fit decays past the exported distance support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants