astropy.github.com icon indicating copy to clipboard operation
astropy.github.com copied to clipboard

Update affiliated package listing script to get from pyopensci

Open eteq opened this issue 1 year ago • 3 comments

This is a follow on from #573 - in the interest of rolling that out quickly I think we should deal with fixing the script separately from updating the infoormation on the affiliated page.

So this issue is to update the old script, or (more likely)... make a new one to pull the listing from pyopensci's listing.

A few unanswered questions:

  • How to keep the "legacy" affiliated packages that may not be in pyopensci?
  • How to separately manage the coordinate packages since they have to be accepted separtely
  • Can we ditch the old script completely eventually? Or will it need to stay active long-term to support the above?

eteq avatar Mar 18 '24 16:03 eteq

When I thought about freezing the "legacy affiliation" page, I didn't think about the case where someone of a listed package just want to update, say, the website or maintainer list, while still being ok with it under "legacy". Should we allow such a change after the freeze? If so, maybe we have to keep the script to make it easier on them.

pllim avatar Mar 18 '24 19:03 pllim

I think we should allow changes like that, e.g. changes in the website might occur at no fault of the maintainer, e.g. if their hosting provider stops supporting something. We would not trigger a re-review for that, so we should not make them go through pyopensci just for that.

However, such changes are rare. If we have good reason to get rid of the script (but if it works and nothing is broken, we might as well leave it), editing pure html would be acceptable for changes like that.

hamogu avatar Mar 18 '24 21:03 hamogu

@eteq , I found two issues here that you might be interested in:

  • pyOpenSci/software-peer-review#268
  • pyOpenSci/software-peer-review#276

pllim avatar Apr 29 '24 16:04 pllim

It seems the Packages list does not highlight it yet, but the metadata already contains the relevant information for e.g. ZodiPy identifying our packages now – parse

partners:
    - astropy

dhomeier avatar May 13 '24 16:05 dhomeier

Right now, I can think of 3 different things we can do:

  1. Use jekyll like https://github.com/pyOpenSci/pyopensci.github.io (e.g., https://github.com/pyOpenSci/pyopensci.github.io/pull/207)
  2. Modify https://github.com/astropy/astropy.github.com/blob/main/js/functions.js to parse YAML from https://github.com/pyOpenSci/pyopensci.github.io/blob/main/_data/packages.yml with the partners:\n\t- astropy filter Derek mentioned above.
  3. Write out the HTML by hand.

@eteq ?

pllim avatar May 13 '24 18:05 pllim

I'm not a fan of 1 because I think that could be a major project with unforeseen consequences. 2 feels much more maintainable to me than 3.

I didn't realize https://github.com/astropy/astropy.github.com/issues/579#issuecomment-2108190251 until now, though, which makes me think this might be a lot easier (i.e. "less javascript"...) than I was earlier thinking, so thanks @dhomeier !

eteq avatar May 13 '24 19:05 eteq

this might be a lot easier (i.e. "less javascript"...) than I was earlier thinking

@eteq , can you please elaborate?

pllim avatar May 14 '24 13:05 pllim

Please review https://github.com/astropy/astropy.github.com/pull/590 . Thanks!

pllim avatar May 17 '24 23:05 pllim