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

D47crunch name #14

Open
timpol opened this issue Jul 20, 2023 · 2 comments
Open

D47crunch name #14

timpol opened this issue Jul 20, 2023 · 2 comments

Comments

@timpol
Copy link

timpol commented Jul 20, 2023

Mathieu now that D47crunch also handles D48 crunching, would you consider changing the name to something mass neutral, like clumpy? I.e.

import numpy as np
import clumpy as cp

Or would this be a terrible idea since D47crunch is mentioned quite a bit in your uncertainty propagation paper, and probably elsewhere?

@mdaeron
Copy link
Owner

mdaeron commented Jul 20, 2023

Hi Tim, that's indeed something to consider.

Con(s):

  • Backwards-compatibility. Changing a library's name will break existing scripts. General practice is to avoid such drastic changes unless there is a compelling reason to do so.

Pro(s):

  • What you mentioned: the library now handles more than Δ47. One may also argue that not many people would use it for Δ48 or Δ49 and not also for Δ47.
  • Discoverability: people might rule out using the library if they assume that it only handles Δ47? I try to address this prominently in the readme and "about" tab.

Also, the best suitable name is not obvious to me:

  • clumpy covers much more than Δ47/48/49. I don't want to piss off the methane people for instance.
  • D4xcrunch is pretty arcane, I suspect many would not get what it's about without going into the readme.
  • co2clump et al. is possible, but there are many similar packages out there and I'm not sure what names they all use.
  • FWIW, the first ever version of this was called rambaldi. John E. loved it: it's nerdy and arcane. Milo Rambaldi is a fictional, 15th-century character from Alias, who has a fetish for the number 47 and uses it to encrypt various bits of secret code.

Ideally, any name change would still let people use import D47crunch for backwards-compatibility. I can look into how it's usually done. Turns out this is probably easy to achieve.

@timpol
Copy link
Author

timpol commented Jul 21, 2023

Are the methane people not familiar with how to do a pull request? :)

Those are all good and interesting points. I'm not familiar with Alias, but I can see why rambaldi would please someone who is!

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

2 participants