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

Rename the package #7

Closed
ocefpaf opened this issue Mar 13, 2015 · 6 comments
Closed

Rename the package #7

ocefpaf opened this issue Mar 13, 2015 · 6 comments

Comments

@ocefpaf
Copy link
Member

ocefpaf commented Mar 13, 2015

@ChrisBarker-NOAA let s move the discussion here.

Maybe "cf_units" it's all about CF, but theoretically only used by Iris, rather than the other way around. But I don't care, really. Good to have "units" in the name, so people will find it.

I like cf_units. @rsignell-usgs what do you think? However, it has to be a name that iris devs are fine with. The goal is to aggregate efforts in one package instead of fragmenting it.

@ocefpaf : maybe udinits2 could be built-in, rather then kept as a separate package. It would make it harder to build, but then it would be only one package to manage.

Right now, we have udunits2 on conda for Linux, Windows and Mac (thanks to you 😄 ), so that is not a big issue. Also, my Linux packager side is biased against it... I like to see small packages with proper dependencies 😁

@rsignell-usgs
Copy link

I like iris_units or cf_units equally well. Let's see what @rhattersley and @pelson have to say?

@pelson
Copy link
Member

pelson commented Mar 13, 2015

I don't think it needs to be named iris_* - I like cf_units as a descriptive name - @rhattersley tends to be pretty creative with names if you want it to be non-descriptive 😉.

The goal is to aggregate efforts in one package instead of fragmenting it.

I whole-heatedly buy into this perspective. There is room under the scitools organisation for this to be hosted (i.e. github.com/scitools/cf_units) if that is where you want to go - I'd happily add my support for @ocefpaf to be given core developer status of that repository along with the existing @SciTools devs.

Thoughts?

@rhattersley
Copy link
Member

👍 for cf_units 😄

... with the proviso that this package is intending to provide CF units support, as opposed to just a Python wrapper for UDUNITS-2. For example, it might aspire to support level, layer, and sigma_level in some fashion, or ensure that "scale factors and offsets [are] not supported".

(It might be worth seeing if the corresponding functionality from cf-python can be folded in and shared. @hasselldc?)

@ocefpaf
Copy link
Member Author

ocefpaf commented Mar 13, 2015

cf_units it is. (For a moment there I thought @rhattersley would suggest for Biggus D. friend N. Maximus!)

@ocefpaf to be given core developer status of that repository along with the existing @SciTools devs.

I'd love to see this at github.com/scitools/cf_units.

@pelson I don't think I have the coding knowledge nor experience to be a @SciTools dev, but I will do my best to maintain and move cf_units forward.

For example, it might aspire to support level, layer, and sigma_level in some fashion, or ensure that "scale factors and offsets [are] not supported".

Make sense #8.

@rhattersley
Copy link
Member

I'd love to see this at github.com/scitools/cf_units.

@ocefpaf - Following the GitHub help, I've sent you an invite to join the SciTools org. That should enable you to transfer the repo if you so wish.

For a moment there I thought @rhattersley would suggest for Biggus D. friend N. Maximus!

Drat! I didn't think of that! 😉

@ocefpaf
Copy link
Member Author

ocefpaf commented Mar 14, 2015

That should enable you to transfer the repo if you so wish.

Done.

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

No branches or pull requests

4 participants