Skip to content

DOC: Move sphinx-gallery examples out of core (this) repository #16175

Closed

Description

@pllim

Motivation: These examples have not been updated nor expanded for a very long time. We have Learn Astropy for these sorts of tutorials. Removing these would free us from sphinx-gallery and speed up doc build (xref #16161).

Cons: Would be impossible to test these examples in PRs (using the RTD job). (Though some people see this as "pros" instead because they do not want the overhead.)

At the 2024-03-07 dev telecon, there was an idea to move these examples to their own repo(?) and only build them against stable astropy (maybe via a cron job or as part of the release process).

Activity

  1. mhvk commented on Mar 7, 2024

    @mhvk
    Contributor

    I like the idea of moving the examples out - but having some way that we are sure to test them on cron or before a new release.

  2. saimn commented on Mar 8, 2024

    @saimn
    Contributor

    I don't know for the coordinates ones, but for the io examples which are mostly io.fits, most of the examples are already in the docs. Would not be too difficult to integrate the missing ones.
    Arguably it's a bit more difficult to find how to do basic things in the docs since examples there are a bit everywhere. Maybe the should a Quickstart/How to do basic things page with the most common examples.

  3. pllim commented on Mar 8, 2024

    @pllim
    MemberAuthor

    @saimn , there is already an extensive "Getting Started" section. Won't adding a similarly named "Quick Start" be confusing?

    https://docs.astropy.org/en/latest/io/fits/index.html#getting-started

  4. saimn commented on Mar 11, 2024

    @saimn
    Contributor

    @pllim - Right, and most examples are already there. The one that is missing in the docs is https://docs.astropy.org/en/latest/generated/examples/io/skip_create-large-fits.html#sphx-glr-generated-examples-io-skip-create-large-fits-py but this should probably go somewhere more hidden, maybe in the FAQ. It's not really an example for newcomers :)

  5. pllim commented on Mar 13, 2024

    @pllim
    MemberAuthor

    @saimn , turns out the FITS landing page link to most of them, except one. How does #16194 look?

  6. pllim commented on Mar 18, 2024

    @pllim
    MemberAuthor

    Simon and I tackled io in #16194 . When I have time (read: procrastinate), I will go for coordinates! 🔥

  7. GuilhermePonce commented on Oct 15, 2024

    @GuilhermePonce

    Hey guys, I wanna try to solve this issue.

  8. pllim commented on Oct 28, 2024

    @pllim
    MemberAuthor

    Thanks for your interest, @GuilhermePonce ! However, I already have a plan for it. That said, we have many other issues you could contribute to. Please see https://www.astropy.org/contribute.html#contribute-code-or-docs .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions