astropy / astropy-APEs

A repository storing the Astropy Proposals for Enhancement.
Other
35 stars 37 forks source link

Is APE 13 still considered "current"? (Yes, but not the viz tool part) #83

Closed weaverba137 closed 1 month ago

weaverba137 commented 1 year ago

I had suggested this as a Coordination meeting topic on astropy/astropy-project#255. But maybe we can save time by simply asking the question: is APE 13 still considered the current guidance on spectroscopy code development, or are substantial revisions needed? Has it been superseded by another document?

I want to be able to point people from outside the Astropy community to that document or its equivalent. These people may be potential contributors to code development, so it's more than just for the purpose of satisfying a passing interest.

As the contributors to APE 13: @kelle @eteq @pllim

pllim commented 1 year ago

For the record, I never contributed to that APE (if I did, I have no recollection and probably not in a significant way). The authors on the record are @crawfordsm , @nmearl , @keflavich , and @eteq .

weaverba137 commented 1 year ago

Fair enough, but at the top of APE 13 those are the three contributors.

dhomeier commented 1 year ago

That would probably be because the actual document had been developed largely as APE_specutil.

crawfordsm commented 1 year ago

Just dropping by to say that it might be worthwhile to bring in @tepickering for comment as to the status of specreduce.

tepickering commented 1 year ago

APE 13 still looks accurate w.r.t. its mentions of specreduce and its scope. development of specreduce has finally picked up some steam and significant progress is being made.

pllim commented 1 year ago

At Astropy Coordination Meeting 2023, @eteq said APE 13 would be "current" again if we apply a minor update, particularly removing mention of visualization tool.