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

suggestion: move log/exp combinations from StatsFuns.jl #98

Closed
tpapp opened this issue Jun 17, 2018 · 4 comments
Closed

suggestion: move log/exp combinations from StatsFuns.jl #98

tpapp opened this issue Jun 17, 2018 · 4 comments

Comments

@tpapp
Copy link

tpapp commented Jun 17, 2018

Maybe these functions could be moved to SpecialFunctions.jl.

@ararslan
Copy link
Member

What's wrong with having them in StatsFuns?

@tpapp
Copy link
Author

tpapp commented Jun 17, 2018

While they are used in statistics (among many other fields), these functions are not per se statistical.

Also, because of JuliaStats/StatsFuns.jl#20, I was under the impression that that package is not for end users, while logistic, softmax, etc are useful functions on their own.

Anyway, this is just a suggestion that I thought would bring up, when revisiting the discussion for the PR above which suggested moving them to StatsBase.jl — I think SpecialFunctions.jl would be more appropriate.

@ararslan
Copy link
Member

Yeah perhaps. (Btw, I'm not saying it's a bad suggestion, I was just wondering what your motivation was for making it. 🙂)

The plan that was discussed a while back was to move distribution-specific code in StatsFuns to Distributions, then move the rest to StatsBase. I don't have a huge preference for where functions like that end up living.

@tpapp
Copy link
Author

tpapp commented Jun 18, 2018

Thinking about this, maybe SpecialFunctions.jl is not the right place for them either. I think they should go to their own package, and opened a discussion in JuliaStats/StatsFuns.jl#46.

@tpapp tpapp closed this as completed Jun 18, 2018
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