astropy-APEs icon indicating copy to clipboard operation
astropy-APEs copied to clipboard

Remove and refactor visualization references in APE 13

Open weaverba137 opened this issue 1 year ago • 9 comments

This PR closes #83. Previously, APE 13 referred to specviz which is now part of a separate project. Suggestions for separate spectrum visualization projects to mention as examples (but not part of APE 13) are welcome.

weaverba137 avatar Jun 14 '24 10:06 weaverba137

@weaverba137 , thanks. I see that this is still in draft form. Do you plan to revisit and take it out of draft?

pllim avatar Jul 08 '24 16:07 pllim

Thank you for the reminder. I'm on vacation, but I will hopefully finish this next week.

weaverba137 avatar Jul 08 '24 17:07 weaverba137

@pllim, this is ready for review now.

weaverba137 avatar Jul 19 '24 16:07 weaverba137

Thanks! I posted on Astropy Slack #spectroscopy channel and put this on CoCo agenda.

pllim avatar Jul 19 '24 17:07 pllim

looks good to me. thanks!

tepickering avatar Jul 19 '24 19:07 tepickering

Looks good to me too.

rosteen avatar Jul 22 '24 14:07 rosteen

Looks good to me.

kelle avatar Jul 22 '24 16:07 kelle

This document mentions the "Spectroscopic Coordinating Committee", I think that as of now, this is identical to the spectroscopy group that meets once a month and uses the #spectroscopy channel on Slack. In any case, perhaps we should identify this committee specifically.

weaverba137 avatar Jul 25 '24 19:07 weaverba137

This document mentions the "Spectroscopic Coordinating Committee", I think that as of now, this is identical to the spectroscopy group that meets once a month and uses the #spectroscopy channel on Slack. In any case, perhaps we should identify this committee specifically.

Good catch. I'll put this on the CoCo agenda.

kelle avatar Jul 25 '24 21:07 kelle

Any progress on the "committee" stuff? What is holding this up now and how can we push it forward? Thanks!

CoCo meeting notes has a note that says "Send an email - Erik – Did Erik send that email?" but I don't remember what this email is supposed to be about. Do you remember, @eteq ?

pllim avatar Aug 05 '24 19:08 pllim

I read that as the email of notification for the 2 week comment period, which has not been sent yet as far as I can see.

For the committee it is not clear to me what actionable items this involves for CoCo. The spectroscopy group and channel seems to reflect the status quo described as

although this committee has been operating in an informal manner prior to this APE

so maybe that comment could include a mention of #spectroscopy channel, dev calls and workshops, but it does not really match the description of a Coordination Committee Patterned after the Astropy Coordinating Committee in the APE.

dhomeier avatar Aug 20 '24 00:08 dhomeier

The content of APE should be general enough that we don't have to come back and modify it again every time spectroscopy people decide to change how they communicate to each other (Slack channel, working groups, committee, etc). Since a real committee (with charter and stuff) never really formed, should we make that text more generic?

pllim avatar Aug 21 '24 01:08 pllim

To push things forward, I have started a two-week comment period (2024-08-23 till 2024-09-06): https://groups.google.com/g/astropy-dev/c/m7v7gxm9QlI/m/xJ6vZkBQBAAJ

pllim avatar Aug 23 '24 15:08 pllim

It's not my impression that they have continuously been changing structures, rather the APE as it was passed lays out a plan for a future (more formalised) structure. If everything works better, or already good enough, with the current organisation, it might indeed be a good idea to adopt the APE accordingly – I think no other sub-field within the Astropy ecosystem has a similar governing body, though probably also no other has been so much and consistently in the focus of roadmap etc. However that is a change not really related to this PR's original topic.

dhomeier avatar Aug 23 '24 16:08 dhomeier

I'm happy with this. I just want to note that most of the discussion in this thread is about removing specviz, but the added focus on IO isn't mentioned prominently. I think that's just as important as the removal of specviz (and is a very good thing!)

(new text is "supporting IO on data files used in the community")

keflavich avatar Aug 23 '24 17:08 keflavich

@keflavich, I like your idea. Could you suggest a place to insert that text?

weaverba137 avatar Aug 23 '24 17:08 weaverba137

That text is already in the PR, I just meant to highlight it in the Issue here, since I hadn't seen it mentioned before

keflavich avatar Aug 23 '24 17:08 keflavich

A quick summary of some of the history here: The original authors of this APE were the "spectroscopic coordinating committee" for a while. We met for some time, and I think there was even an entry (or at least a PR to add it) on https://www.astropy.org/team.html for the group. But with time the group sort of dissolved for various reasons - a mix of "peoples' jobs changed" and "it was hard to drum up community interest, which was very demotivating". And that is what prompted the more recent push to try to coordinate with different relevant interest groups. @keflavich might or might not have a bit more to add on that?

(None of that needs to be in this APE, I'm just providing it for context because several people in the spectroscopic meeting last week didn't know this part.)

To me I'd say it's an equally valid choice to either declare the group that meets regularly the "new" spectroscopic coordinating committee, or just remove mention of it because the idea didn't really pan out the way we'd hoped. (In which case the newer group should still keep doing what it's doing, it just doesn't need to be the body referenced in this APE). I personally am fine with letting the new committee decide (we talked about it in the meeting last week, and my sense was "ambivalence", although @weaverba137 or @kbwestfall might feel differently?)

eteq avatar Sep 03 '24 15:09 eteq

@eteq, thank you for the write-up. I agree that last week we neither endorsed nor rejected the idea of becoming "the committee" as referenced in the APE. However I am not ambivalent about changing the language in this PR to reflect the present situation. Maybe that means calling it an ad hoc committee or a similar tweak. I would welcome suggestions.

weaverba137 avatar Sep 03 '24 16:09 weaverba137

On the coordinating committee, I read through the doc to summarize what it articulates as the core responsibilities of the "Spectroscopic Coordinating Committee." (SCC) Here's my summary:

  1. Assists the AstroPy Coordinating Committee (CoCo) to consider and consolidate overlapping functionality between spectroscopy packages seeking astropy-affiliated status. (The doc also specifically states that one member of the SCC is also a member of the CoCo.)
  2. Oversees effort related to spectroscopy within the astropy ecosystem.

Does this agree with what others take from the doc? I would argue that the current SpectroscopyDev group doesn't really do either of these things. That's not to say that we couldn't, but it's a different scope compared to our current mode of largely just checking in with one another to discuss progress --- or lack thereof :) --- and next steps. I could see someone arguing we do 2, but "oversee" feels like too strong of a word to me.

If the doc can reflect both the current situation and something to work toward, I suggest we leave most of the language as is. I would propose that the SpectroscopyDev group that currently meets monthly can assume the role, with some minor changes.

  • For 1, I would add "as needed." I.e., if the CoCo wants/needs it, we can help.
  • We could also potentially remove the statement that a member of the SCC is also a member of the CoCo; i.e., I'm not sure this needs to be formalized.
  • And for 2, I would change "oversee" to "coordinate"; specifically see line 105.

kbwestfall avatar Sep 03 '24 23:09 kbwestfall

Thank you @kbwestfall, I've corrected the typos you found.

weaverba137 avatar Sep 04 '24 17:09 weaverba137

@pllim, the two week period has passed.

weaverba137 avatar Sep 10 '24 17:09 weaverba137

Thanks for the ping, @weaverba137 ! This is on CoCo agenda. Thanks for your continued patience.

pllim avatar Sep 10 '24 19:09 pllim

At the Spectroscopy tag up today, we agreed that we are willing to wait for guidance from the CoCo--and possibly a separate PR redefining "Spectroscopic Coordinating Committee"--rather than immediately assuming that role.

Meanwhile, informally, the spectroscopy tag up group could discuss strategic planning on an as-needed basis.

weaverba137 avatar Sep 26 '24 19:09 weaverba137

Alright, the CoCo agrees with the above, so I am merging this. Thanks @weaverba137 for carrying this over the finish line!

eteq avatar Sep 26 '24 19:09 eteq

Note I can't update the zenodo entry at merge-time as usual, due to access rights issues in Zenodo. Once we sort that out will leave a message here.

eteq avatar Sep 26 '24 19:09 eteq

Follow up issues:

  • #110
  • #111

pllim avatar Sep 26 '24 19:09 pllim