Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Affiliated packages with pyOpenSci: Link to packages under review? #580

Closed
pllim opened this issue Mar 21, 2024 · 19 comments · Fixed by #595
Closed

Affiliated packages with pyOpenSci: Link to packages under review? #580

pllim opened this issue Mar 21, 2024 · 19 comments · Fixed by #595

Comments

@pllim
Copy link
Member

pllim commented Mar 21, 2024

This is a follow up of:

Would it be useful to add a link to https://github.com/pyOpenSci/software-submission/issues?q=is%3Aissue+is%3Aopen+label%3Aastropy so people can see what are currently under review at pyOpenSci? Unless @lwasser has a better way?

cc @dhomeier @hamogu

@hamogu
Copy link
Member

hamogu commented Mar 21, 2024

We can do that, but I'm not sure it's required. I'm not going to stop anyone from adding a link, but I can also think of more important things to do with our time.

@lwasser
Copy link
Contributor

lwasser commented Mar 21, 2024

That is the best way @pllim
For reference: i'm working on a build that will allow us to update review status in real time on the website (creating a yaml file or json). However, it will take some time for that to be complete, tested working etc.

@pllim
Copy link
Member Author

pllim commented Mar 22, 2024

Okay, I'll just leave this issue open for a bit and see. It is certainly not urgent but if people start looking at our updated Affiliated page and get alarmed why the list is empty, then maybe we can throw this link in then.

@bsipocz
Copy link
Member

bsipocz commented Mar 22, 2024

Okay, I'll just leave this issue open for a bit and see. It is certainly not urgent but if people start looking at our updated Affiliated page and get alarmed why the list is empty, then maybe we can throw this link in then.

That is indeed alarming. Are packages lost their affiliated status with entering in this collaboration? My impression was that they didn't, so banishing them now to a different page and call them legacy is not exactly a good move. Could the actual affiliated package HTML be a merge of the two sites with two tables rather than totally separate?

@hamogu
Copy link
Member

hamogu commented Mar 22, 2024

No. Packages have been affiliated before, are still affiliated now; they are not loosing this status now [*]. However, we do want to present the lists separately - some packages might even appear on both lists at some point.

*: However, we've always said that there should be an occasional re-review of packages with packages that become unmaintained loosing there affiliated status at some point. It's just that in practice the affiliated editors have never gotten to that point. I hope that, in collaboration with PyOpenSci, we'll get to do that in the coming years (starting with the oldest package at a rate of a few per year, so nobody gets reviewed more than once every 5-10 years.

@hamogu
Copy link
Member

hamogu commented Mar 22, 2024

Maybe the best solution is just small rewording from "is currently empty" to "The list of affiliated packages is quite long and we thus split it in packages that have been reviewed by PyOpenSci starting March 2023 (below) and the packages that became affilated before MArch 2024 (list of 48 packages AT THIS LINK)"?

@bsipocz
Copy link
Member

bsipocz commented Mar 22, 2024

list of 48 packages AT THIS LINK)

But those lived under the affiliated link for so many years, it kind of should be the other way around. Or both to be listed on the same, usual, link.

@bsipocz
Copy link
Member

bsipocz commented Mar 22, 2024

(And re-reviewing the status is totally different than sending them through a new process they haven't signed up for)

@pllim
Copy link
Member Author

pllim commented Mar 22, 2024

If "legacy" reflects negatively, we could change it to "pre_ape22" or something. Open to suggestions but I do want to keep it on a separate page to avoid clutter... #573 was opened for a while and posted on Slack and astropy-dev for comments.

@pllim
Copy link
Member Author

pllim commented Mar 22, 2024

Also maybe continue this concern on a new issue as it seems to be getting out of topic here? Thanks!

@bsipocz
Copy link
Member

bsipocz commented Mar 22, 2024

separate page to avoid clutter.

If having ~50 packages listed as clutter, then sure, but having an ecosystem was kind of seen as a positive before so I still don't see why the banish of those pre-ape22 packages is seen as a good thing in favour of a currently empty page.

@hamogu
Copy link
Member

hamogu commented Mar 22, 2024

(And re-reviewing the status is totally different than sending them through a new process they haven't signed up for)

No, it's not. It's slightly different, but not much since the process is almost the same and the requirements are almost the same. But that's why we went through the APE process (incl notices on slack and astrop-dev) so that people who disagree had ample opportunity to comment. There is a whole section on "re-review" in APE 22.

@hamogu
Copy link
Member

hamogu commented Mar 22, 2024

Having ~50 affiliated packages is still good! However, how it's displayed on the website has not evolved since we had 5 packages or so. A website that's easily readable with 5 package is not with 50. We could have split the page when we reach 30 affiliated packages or when we reached 40 affiliated packages or waited till we reach 100. There is obviously no exact number - but as the number grows (and continues to grow) we need to adjust how it's displayed on the page.

We could have done that independent of the pyopensci and APE22, but it just happens to be a good opportunity now because we need te edit the text anyway.

@pllim
Copy link
Member Author

pllim commented Mar 22, 2024

Just so everyone knows if I go quiet, I am about to take some time off. I will come back and address replies when I return. Thanks for your patience!

@hamogu
Copy link
Member

hamogu commented Mar 22, 2024

That does not mean that text is perfect - it can obviously be changed - but there is a reason that listings on any website or listing (google, MAST, Chandra search form, ADS) typically output a limited number of results per page. We don't want to be that complicated, but splitting in some way makes sense to me.

@bsipocz
Copy link
Member

bsipocz commented Mar 22, 2024

Then a rephrasing is very timely, as currently the page states, before the empty table that "The following table lists all currently registered affiliated packages." all the while you claim that packages previously in the list didn't lose their affiliated status. Those are contradicting. So call those in the empty table affiliated with pyopensci, or similar, rather than adding extra, negative adjectives to those already affiliated.

And frankly, the ape was about the collaboration and offloading the editorial process, not that you banish the old listing. I fully agreed with the former and thus didn't follow the rest of the very very lengthy APE process, so now ending up with a practically hidden "legacy" page is not that great experience. And, no neither google, or chandra, or ads are returning empty results as opposed to ~50 like in this case and are listing all the stuff if you ask for it rather than dumping it into a hidden page linked under a here at best.

@pllim
Copy link
Member Author

pllim commented Apr 3, 2024

I will propose a follow-up PR to address the concerns that started around #580 (comment) . Thank you for your patience.

Update: See #582

@pllim
Copy link
Member Author

pllim commented May 22, 2024

Back to the original topic, ZodiPy has been accepted into pyOpenSci, are there other packages under review, @dhomeier or @hamogu ?

@pllim
Copy link
Member Author

pllim commented May 22, 2024

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

Successfully merging a pull request may close this issue.

4 participants