Skip to content

Update to use scenario=flight in make_timeline.py#94

Merged
jeanconn merged 11 commits intomasterfrom
kadi-scenario
Sep 26, 2025
Merged

Update to use scenario=flight in make_timeline.py#94
jeanconn merged 11 commits intomasterfrom
kadi-scenario

Conversation

@jeanconn
Copy link
Copy Markdown
Contributor

@jeanconn jeanconn commented Jul 15, 2025

Description

Update to use scenario=flight for make_timeline.py.

Replan central runs on HEAD and always has access to up-to-date commands in /proj/sot/ska, so there's no reason to do web queries (that frequently fail and toss warnings into the arc logs).

Interface impacts

Functional testing

No functional testing.

@jeanconn jeanconn changed the title Update to use KADI_SCENARIO=flight for make_timeline.py Update to use scenario=flight in make_timeline.py Aug 21, 2025
@jeanconn
Copy link
Copy Markdown
Contributor Author

jeanconn commented Aug 21, 2025

Given the scope, I'm wondering if we can skip end-to-end functional testing on this one and just verify by inspection the change is formatted correctly and get the functional testing as part of #95 .

@jeanconn jeanconn marked this pull request as ready for review August 21, 2025 15:58
@jeanconn jeanconn requested a review from taldcroft August 27, 2025 19:47
@jeanconn
Copy link
Copy Markdown
Contributor Author

This has now been functionally tested in #95

@jeanconn jeanconn requested review from taldcroft and removed request for taldcroft September 26, 2025 11:58
@jeanconn
Copy link
Copy Markdown
Contributor Author

Technically, I think this still needs approval before I should cut a release and to the FSDS.

Copy link
Copy Markdown
Member

@taldcroft taldcroft left a comment

Choose a reason for hiding this comment

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

Looks good.

Update paths for MTA radiation reorg
@jeanconn jeanconn merged commit 4a5919c into master Sep 26, 2025
2 checks passed
@jeanconn jeanconn deleted the kadi-scenario branch September 26, 2025 14:55
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