Skip to content

Conversation

@jkrick
Copy link
Contributor

@jkrick jkrick commented Jan 22, 2026

closes #217

This PR should

along the way, I have

  • renamed "Special Topics" to "Techniques & Tools"
  • made the notebook descriptions very short in preparation for cards instead of lists on the landing pages.

I did not

  • rename the notebooks in line with their short titles, however tempting this is
  • re-organize the repo directory structure to bring it in line with the toc, however tempting this is

@jkrick jkrick self-assigned this Jan 22, 2026
@jkrick
Copy link
Contributor Author

jkrick commented Jan 22, 2026

@bsipocz I am adding short titles to the irsa tutorials per #217 . Should I also change the names of the actual .md tutorial files to match those short titles. Would make sense to me, but I don't see the whole picture. I put this draft up as an example. So in this case, I would recommend re-naming spherex_intro.md to data_overview.md to match the short title I chose, and also for the other tutorials. I didn't choose "spherex intro" as the short title (which would have simplified this) because I think having a table of contents with "spherex" as the main header and then under that something that repeats the term "spherex" is not a good idea.

@jkrick jkrick changed the title adding shorttitle and landing page for spherex adding short_title and landing page for spherex Jan 22, 2026
@jkrick
Copy link
Contributor Author

jkrick commented Jan 22, 2026

@bsipocz @troyraen Also, the directory structure of irsa-tutorials doesn't match the table of contents structure. Should I change that to be in line with each other? ie., in the table of contents on https://caltech-ipac.github.io/irsa-tutorials/ we have a "WISE" header, but there is no wise directory under irsa-tutorials/tutorials/ This is because we changed the table of contents to be mission specific ~1 year ago. How deep into this can of worms do you want me to dig? What other monsters will I unleash if I keep digging?

Comment on lines 10 to 16
SPHEREx IRSA data releases include weekly [Quick Release spectral image products](https://caltech-ipac.github.io/spherex-archive-documentation/spherex-data-products/) (multi-extension FITS files containing calibrated near-infrared surface brightness, variance, flags, modeled backgrounds, PSFs, and wavelength WCS) along with ancillary calibration and metadata files such as gain matrices, dark current maps, solid angle pixel maps, and detailed spectral WCS products for each detector.

- [Data Overview](https://caltech-ipac.github.io/irsa-tutorials/spherex-intro/) - Introduces the SPHEREx mission data available at IRSA, explaining the structure of the data products and how to identify which files and formats are appropriate for different science use cases.

- [Spectral Image Cutouts](https://caltech-ipac.github.io/irsa-tutorials/spherex-cutouts/) - Demonstrates how to generate and work with spatial and spectral cutouts from SPHEREx data, enabling users to extract subsets of the all-sky survey tailored to specific targets or regions.

- [PSF Models](https://caltech-ipac.github.io/irsa-tutorials/spherex-psf/) - Explains how SPHEREx point spread function (PSF) information is organized and accessed, and shows how PSF models can be used to interpret, model, or forward-simulate SPHEREx observations.
Copy link
Member

Choose a reason for hiding this comment

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

Don't use full render URLs, but internal references. The labels you deleted below should be the ones

name: python3
---

(spherex-cutouts)=
Copy link
Member

Choose a reason for hiding this comment

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

Keep this

name: python3
---

(spherex-intro)=
Copy link
Member

Choose a reason for hiding this comment

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

Keep this

@bsipocz bsipocz added the html rendering / skip testing Rendering related issues/PRs. Skips tests in PRs. label Jan 22, 2026
@bsipocz
Copy link
Member

bsipocz commented Jan 22, 2026

@bsipocz I am adding short titles to the irsa tutorials per #217 . Should I also change the names of the actual .md tutorial files to match those short titles. Would make sense to me, but I don't see the whole picture. I put this draft up as an example. So in this case, I would recommend re-naming spherex_intro.md to data_overview.md to match the short title I chose, and also for the other tutorials. I didn't choose "spherex intro" as the short title (which would have simplified this) because I think having a table of contents with "spherex" as the main header and then under that something that repeats the term "spherex" is not a good idea.

OK, so this will be a multitude of a few things.

  1. no need to rename the files, the filename and short title doesn't have to be connected, the short title is just the text that will show up in the TOC on the left hand side

    • Jupyter Lab/hub/notebook apparently doesn't like the myst metadata. I don't yet have a bug report for this, but it seems that we either be very careful with committing any changes to the metadata after editing notebooks in JH (Use query_ssa() for Euclid tutorials 3 & 5 #216 (comment)) or find another solution

2*) with the very latest mystmd version released yesterday it is now possible to assign short titles in the toc file. So, I think we should do this as it is both more correct (that has effect on the TOC/rendering only not on the notebook itself) and also works around the bug mentioned above.

  1. If you want to rename files for any reasons (e.g. you would find it easier to navigate and find them in JH) then we can do that. I have no opinions of what they should be called (but having unique names would help); but the caveat is that we need to make sure they are renamed everywhere -- in the manifest file, and in any cross references where we may still use file names rather than internal labels.

@jkrick
Copy link
Contributor Author

jkrick commented Jan 22, 2026

Jupyter Lab/hub/notebook apparently doesn't like the myst metadata. I don't yet have a bug report for this, but it seems that we either be very careful with committing any changes to the metadata after editing notebooks in JH (#216 (comment)) or find another solution

ugh, really? I looked at these, and the only change I saw is the one I wrote with short_title. Which I did after following #206 Unfortunately this is a learning process of which things are problems and which aren't.

2*) with the very latest mystmd version released yesterday it is now possible to assign short titles in the toc file. So, I think we should do this as it is both more correct (that has effect on the TOC/rendering only not on the notebook itself) and also works around the bug mentioned above.

can you tell me how to do this? or an example?

@bsipocz
Copy link
Member

bsipocz commented Jan 22, 2026

ugh, really? I looked at these, and the only change I saw is the one I wrote with short_title. Which I did after following #206 Unfortunately this is a learning process of which things are problems and which aren't.

This is all good, the rendering and everything should be fine, but if you open these in jupyter and run them I believe the short_title entries will get deleted. That happened in your euclid PR at least.

@bsipocz
Copy link
Member

bsipocz commented Jan 22, 2026

2*) with the very latest mystmd version released yesterday it is now possible to assign short titles in the toc file. So, I think we should do this as it is both more correct (that has effect on the TOC/rendering only not on the notebook itself) and also works around the bug mentioned above.

can you tell me how to do this? or an example?

What you have here will work for now, but going fwd I think we will need to do this instead:
https://mystmd.org/guide/table-of-contents#document-titles

The myst version containing this change has already been released (you need 1.8.0 for this -- released yesterday), but we use jupyter-book (which is a thin wrapper) and it is still in the middle of the release process. So it won't yet work for us here in CI. So keep doing what you do here, you shouldn't be blocked I just mention that we have a solution/workaround that we can switch to in the near future.

@jkrick
Copy link
Contributor Author

jkrick commented Jan 22, 2026

I just mention that we have a solution/workaround that we can switch to in the near future.

Any idea what "near future" means? I don't want to touch this twice, so Im happy to leave it alone if there will be a better solution in the next few weeks.

@bsipocz
Copy link
Member

bsipocz commented Jan 22, 2026

Any idea what "near future" means?

yesterday :) No, I mean the release was planned to be done early this week, and most of it was done yesterday. So I would say, this should all work by next week?
(as I don't see a reason why we shouldn't require the latest release; beside this TOC issue I also expect the new release to solve most of our memory problems on circleCI)

@jkrick jkrick changed the title adding short_title and landing page for spherex adding short_title and landing pages for everything other than Euclid Jan 22, 2026
Copy link
Contributor

@troyraen troyraen 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 working on this; we really need it! This content will be very visible, so please don't merge before soliciting feedback from the full team and iterating until consensus. (I see this is marked as draft, so maybe that's already your plan.) For now, I read through and commented on a couple things that stuck out, but not a detailed review.

the directory structure of irsa-tutorials doesn't match the table of contents structure. Should I change that to be in line with each other?

I think we probably should. But this will be a big change, so I prefer that we touch base about it with the full team first. Then if we decide to move forward, make a PR dedicated to that change rather than mixing it in with adding these landing pages.

@bsipocz
Copy link
Member

bsipocz commented Jan 26, 2026

@jkrick - you can now add "title" to the toc.yml file, which is equivalent to the short_title in the metadata. (See #224 for euclid)

Copy link
Contributor Author

@jkrick jkrick left a comment

Choose a reason for hiding this comment

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

@troyraen I was indeed not finished with this PR, but had put it up as a draft to get some thoughts on directions it needed to go before declaring it ready for review.

I think it is now in a good enough state to be ready for review, so please give me your feedback.

@jkrick jkrick marked this pull request as ready for review January 26, 2026 19:21
Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

I'm leaving a request changes status on this, so we don't accidentally forget to remove the temporarily commit.

@jkrick jkrick changed the title adding short_title and landing pages for everything other than Euclid adding titles to TOC and landing pages for everything other than Euclid Jan 26, 2026
@jkrick
Copy link
Contributor Author

jkrick commented Jan 26, 2026

@bsipocz the "title" in the toc isn't showing up in the Rendering Preview. Those are still showing up with some longer text. What am I missing?

Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

The suggestions should fix the issue

Comment on lines 1 to 3
---
short_title: "WISE"
---
Copy link
Member

Choose a reason for hiding this comment

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

These can be removed now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@bsipocz
Copy link
Member

bsipocz commented Jan 26, 2026

the "title" in the toc isn't showing up in the Rendering Preview. Those are still showing up with some longer text. What am I missing?

Added a commit that should fix it up. TL;DR they should belong for the same entry

@troyraen troyraen self-requested a review January 27, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

html rendering / skip testing Rendering related issues/PRs. Skips tests in PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a landing page for each mission in the TOC

3 participants