Skip to content

Update tutorials to work with current versions#19

Merged
braingram merged 1 commit intoasdf-format:mainfrom
perrygreenfield:fixtut
Aug 5, 2025
Merged

Update tutorials to work with current versions#19
braingram merged 1 commit intoasdf-format:mainfrom
perrygreenfield:fixtut

Conversation

@perrygreenfield
Copy link
Copy Markdown

Changed the installation within the converter tutorials to use pyproject.toml

Fixed various formatting issues.

Removed code that was supposed to restart the kernel from executing a cell, that doesn't appear to work anymore; now the user is instructed to do that through the notebook interface.

Changed the use of a keyword "type" to "type_"

@braingram
Copy link
Copy Markdown
Contributor

Thanks for making these updates. The converter notebooks failed for me.

It looks like they're dumping file in ~ then trying to install it. It failed for me with:

Obtaining file:///Users/bgraham/converter_tutorial/mfconverter
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... error
  error: subprocess-exited-with-error
  
  × Preparing editable metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [106 lines of output]
      /private/var/folders/vk/4ry5_1zd2knfy0f8t63cdcsh0005g6/T/pip-build-env-tsum2il7/overlay/lib/python3.13/site-packages/setuptools/config/_apply_pyprojecttoml.py:75: _MissingDynamic: `readme` defined outside of `pyproject.toml` is ignored.
... truncated ...

The notebook cell it generated won't scroll (aren't notebooks great...) so I'm not sure what all went wrong there. Can we update this to instead use files that are already included in the repository (instead of using %%writefile?

For the other notebooks I had to install:

  • asdf
  • astropy
  • asdf-astropy
  • gwcs
  • matplotlib

to avoid crashes. Is this documented anywhere?

@braingram
Copy link
Copy Markdown
Contributor

I'm going to merge this and address the above issues in follow-up PRs.

@braingram braingram merged commit 368d47f into asdf-format:main Aug 5, 2025
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