Skip to content
This repository has been archived by the owner on Jun 29, 2019. It is now read-only.

rename for 0.4: Distance => Distances? #25

Closed
StefanKarpinski opened this issue Aug 25, 2014 · 5 comments
Closed

rename for 0.4: Distance => Distances? #25

StefanKarpinski opened this issue Aug 25, 2014 · 5 comments

Comments

@StefanKarpinski
Copy link

It's become a bit of a convention to use the plural for package names – freeing up the singular Distance for a type name. In this case, it's not really needed since the mathematical term for a distance is "metric", but it might be good to follow the convention when the ecosystem starts switching over to Julia 0.4.

@lindahua
Copy link
Contributor

I have been considering doing this. Just that not sure whether this change would cause a lot of troubles.

cc: @IainNZ

@StefanKarpinski
Copy link
Author

That's why I'm thinking that the 0.4 transition – which is going to be a big one – would be a good time.

@IainNZ
Copy link
Contributor

IainNZ commented Aug 25, 2014

The best way to do it, I think is:

  1. Duplicate this package to JuliaStats/Distances.jl, so you keep git history.
  2. Tag it as v0.1 (or whatever) of Distances.jl
  3. Tag new release of Distance.jl that prints a warning on load and says to change to Distances.jl

@StefanKarpinski
Copy link
Author

Agreed. That should do the trick.

@lindahua
Copy link
Contributor

Ken-B added a commit to Ken-B/Loess.jl that referenced this issue Aug 27, 2014
see JuliaStats/Distance.jl#25 to remove the warning when `using Gadly`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants