-
Notifications
You must be signed in to change notification settings - Fork 0
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
potential name conflict #2
Comments
Thanks, Roger! Yeah, it's true about the name. We thought about this too, and deep down I sort of knew that I was effectively drafting you (and maybe others) into a conversation to about names when I created the repo. I'll tell you the thoughts that Cody and I had when we initially talked about the name, but please tell me what you think, too. Our thoughts were basically:
On the other hand, just because frequent users of Julia might not be confused by a combination of names like FluxML/Flux.jl and JuliaPhysics/Fluxes.jl (if that existed), that doesn't mean somebody coming from PyTorch to experiment with Flux wouldn't be put off or confused. So I definitely see the other side to it, too. |
Hi, I think it's not about the name convention. It's simply just because there is only one letter difference in the name, which potentially make things very confusing. A user will need to install |
Thanks for making this package, and very nice talk! Just want to notify that there is https://github.com/JuliaPhysics/Lattices.jl already which is for defining lattices.
The name of this package might be a bit misleading given the existence of that package. It'd be easier for users to install this package if the name can be changed to a more distinguishable one.
The text was updated successfully, but these errors were encountered: