Skip to content

ASB-35489: Add MAST Roman Engineering database Search & Download notebook#143

Merged
ttdu merged 10 commits intospacetelescope:roman-prelaunchfrom
sedonaprice:mast-roman-edp
Feb 4, 2026
Merged

ASB-35489: Add MAST Roman Engineering database Search & Download notebook#143
ttdu merged 10 commits intospacetelescope:roman-prelaunchfrom
sedonaprice:mast-roman-edp

Conversation

@sedonaprice
Copy link
Copy Markdown

Closes ASB-35489.

As with #142, @zclaytor and I are creating MAST tutorial notebooks for Roman mission folks & others participating in commissioning/pre-launch activities, so these notebooks just need to live somewhere public.

Thus this tutorial could be included in a new branch (likely the same branch as for #142), or whatever makes most sense.

NOTE that Roman Engineering data products search and downloads are currently restricted to authorized users. Authorization is verified with a MAST API token, which the notebook gets from the user's environment. This will cause the notebook CI to fail. Advice on how to handle this prior to launch is needed.

Includes download example notebook, which includes framework for visualization (not fully debugged, as EDP values are presently only available for one time point).

(Squashed to remove commit with evaluated output).
Edit introductory narrative to conform with current MAST notebooks style guide
@ttdu ttdu changed the base branch from main to roman-prelaunch January 28, 2026 16:15
Copy link
Copy Markdown

@zclaytor zclaytor left a comment

Choose a reason for hiding this comment

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

Aside from the four minor things below, this looks good!

Comment thread notebooks/Roman/Engineering_Database_Retreival/EDB_Retrieval.ipynb Outdated
Comment thread notebooks/Roman/Engineering_Database_Retreival/EDB_Retrieval.ipynb Outdated
Comment thread notebooks/Roman/Engineering_Database_Retreival/EDB_Retrieval.ipynb Outdated
Comment thread notebooks/Roman/Engineering_Database_Retreival/EDB_Retrieval.ipynb Outdated
@sedonaprice sedonaprice requested a review from zclaytor February 3, 2026 16:11
Copy link
Copy Markdown

@zclaytor zclaytor left a comment

Choose a reason for hiding this comment

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

Thanks for your fixes. Everything looks good!

sedonaprice and others added 2 commits February 3, 2026 15:40
Rename folder "Engineering_Database_Retreival" -> "Engineering_Database_Retrieval".

No other changes.
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.

3 participants