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

Added glue as an affiliated package #67

Merged
merged 1 commit into from
Jun 10, 2015

Conversation

astrofrog
Copy link
Member

Based on an email discussion a little while back with @ChrisBeaumont and @perrygreenfield, I think we agreed that glue should be an affiliated package. I think this is a good time to make sure we also understand what the implications of being an affiliated package are. So just to be clear, listing glue as an affiliated package does not exclude having it also be affiliated with other projects (e.g. outside astronomy), correct? In my view, the key sentence is the following (from our website):

... but has requested to be included as part of the Astropy project’s community. These packages are expressing an interest in Astropy’s goals of improving reuse, interoperability, and interface standards for python astronomy and astrophysics packages.

Because glue is also intended for use outside astronomy, so we need to be sure that being affiliated does not mean that glue is 'part' of the Astropy project as such. I guess we could try and make these kinds of points clearer on the website to avoid discouraging people from becoming 'affiliated'?

@wkerzendorf
Copy link
Member

@astrofrog TARDIS also has the goal of reuse, interoperability and interface standards. Would you consider us?

@astrofrog
Copy link
Member Author

Of course! :)

@ChrisBeaumont
Copy link

+1 from me :) Nothing about the current wording makes me too nervous that becoming an affiliated package would somehow corner a project as solely-astronomical.

@perrygreenfield
Copy link
Member

Likewise, I don't think being affiliated means it can't be used in other contexts. It just says that the package adheres to the principle of being compatible with astropy, it's documentation, and tests (though tests with GUIs are a very difficult issue).

@cdeil
Copy link
Member

cdeil commented May 5, 2015

For the end-user, probably the most important aspect of http://www.astropy.org/affiliated/ is a link collection, i.e. to find useful packages. Since it's not possible to list everything there (e.g. healpy, sherpa, ...), maybe a link to http://www.astropython.org/resources would be a good idea?

@astrofrog
Copy link
Member Author

@cdeil - yes, once astropython.org has been revamped (currently the astropython site contains packages we shouldn't really be recommending anymore and the plan for the new astropython site will be to make that kind of distinction clearer)

cc @taldcroft

@astrofrog
Copy link
Member Author

@eteq - I think this should be good to go

@eteq
Copy link
Member

eteq commented May 5, 2015

@astrofrog @perrygreenfield - I'm fine with glue as an affiliated package, but I think perhaps for a larger one like this, which had an independent existence prior to being an affiliated package, a request should be posted to astropy-dev first, as is the at least nominally official procedure (and given a few days for any comments before we merge). I don't expect anyone will object, but I'm a bit uncomfortable "short-circuiting" the procedure we've at least said we're following on the web site.

@astrofrog
Copy link
Member Author

@eteq - I can still send an email to the mailing list about this if you would be more comfortable. I had forgotten that emailing the list was part of the formal process.

@eteq
Copy link
Member

eteq commented May 5, 2015

@astrofrog - yeah, I had too, but then I looked at the web site and that is what we said ;)

I'll be making a PR shortly to update those instructions with the provisional review procedure, so we can re-consider the matter there.

@eteq
Copy link
Member

eteq commented May 5, 2015

The PR I mentioned above is #71 - I suggest moving discussion of whether astropy-dev posting should be mandatory be moved there.

(And @wkerzendorf, absolutely feel free to ask for affiliated status for TARDIS whenever you feel it's appropriate. Although as per the discussion above, definitely post to astropy-dev in parallel to a PR.)

@eteq
Copy link
Member

eteq commented May 7, 2015

Alright, #71 is now merged - so I'd say at least briefly mention this on astropy-dev and give it a few days. I'm pretty sure no one will have any objections, but at the very least this means people will see it.

@eteq
Copy link
Member

eteq commented May 14, 2015

@astrofrog @ChrisBeaumont - just as a reminder, the judgement in #71 is that you should put the request into astropy-dev so that anyone there can comment before we can merge this.

@astrofrog
Copy link
Member Author

@eteq
Copy link
Member

eteq commented May 18, 2015

@astrofrog - Awesome. If there are no significant objections by the end of the week, I think we can go ahead and merge this (the coordinating committee already discussed it, so no need to re-visit if there's not further input).

@astrofrog
Copy link
Member Author

@eteq - can we merge this?

eteq added a commit that referenced this pull request Jun 10, 2015
Added glue as an affiliated package
@eteq eteq merged commit 1250922 into astropy:master Jun 10, 2015
@eteq
Copy link
Member

eteq commented Jun 10, 2015

yes!

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

Successfully merging this pull request may close these issues.

6 participants