Skip to content

Catalog browser on roman#133

Open
cganley1 wants to merge 9 commits intospacetelescope:mainfrom
cganley1:catalog-browser-on-roman
Open

Catalog browser on roman#133
cganley1 wants to merge 9 commits intospacetelescope:mainfrom
cganley1:catalog-browser-on-roman

Conversation

@cganley1
Copy link
Copy Markdown
Collaborator

@cganley1 cganley1 commented Mar 30, 2026

  • Updated link to MAST webpage
  • URL stored as env var
  • Adjusted window sizing and added CSS

We should wait to merge this when the CSB tool is ready for production.

@cganley1
Copy link
Copy Markdown
Collaborator Author

Built and tested on roman-dev, select the jesse-dev image. Working as expected.

msanchezst
msanchezst previously approved these changes Apr 1, 2026
Copy link
Copy Markdown
Collaborator

@msanchezst msanchezst left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

" <iframe\n",
" width=\"100%\"\n",
" height=\"600px\"\n",
" src=\"https://mastdev.stsci.edu/schema_browser/#/Roman/Spectra%20(MOCK)%20V3.3/file_metadata\"\n",
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.

is this going to be updated with the new ENV_VAR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This was notebook output mistakenly committed. I've pushed a new commit with this section cleared.

msanchezst
msanchezst previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Collaborator

@msanchezst msanchezst left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

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.

4 participants