-
Notifications
You must be signed in to change notification settings - Fork 1
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
externally accessible functions #47
Comments
I agree. Also adapting the structure from glmnet is generally a good idea. |
Possibly: Change name away from hdcd? E.g. FastChangepointDetection. |
Also generic functions like e.g. plot and print... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Which functions should be externally accessible. I propose, similar to glmnet, two functions:
I do not like the idea of having one function that does cv if one parameter is missing or similar. Maybe throw a warning, choose lambda according to asymptotic theory, delta = 0.1 and return a list of cps corresponding to different gammas if no params are given. Choose a default method (e.g. glasso) and a default optimizer.
The text was updated successfully, but these errors were encountered: