-
-
Notifications
You must be signed in to change notification settings - Fork 103
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
Comments
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. |
That is the best way @pllim ✨ |
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? |
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. |
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)"? |
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. |
(And re-reviewing the status is totally different than sending them through a new process they haven't signed up for) |
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. |
Also maybe continue this concern on a new issue as it seems to be getting out of topic here? Thanks! |
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. |
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. |
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. |
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! |
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. |
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 |
I will propose a follow-up PR to address the concerns that started around #580 (comment) . Thank you for your patience. Update: See #582 |
Ah, nvm... I see it here. https://github.com/pyOpenSci/software-submission/issues?q=is%3Aissue+is%3Aopen+label%3Aastropy |
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
The text was updated successfully, but these errors were encountered: