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

princurve 2.0.2 #20

Closed
rcannood opened this issue Jun 12, 2018 · 7 comments
Closed

princurve 2.0.2 #20

rcannood opened this issue Jun 12, 2018 · 7 comments
Assignees
Labels

Comments

@rcannood
Copy link

Hello Gavin @gavinsimpson,

princurve 2.0.2 was just pushed to CRAN. I notice your package uses princurve and wish to notify you of the changes so you can update your package accordingly.

I believe what is relevant for you is that get.lam has been renamed to project_to_curve, in order to match the name of the function better with what its purpose is. In addition, the input argument and output variable tag has been renamed to ord.

The function get.lam still exists in the package, for now, as not to break analogue, but get.lam will be deprecated on July 1st and later on removed.

If you have any remarks or questions, feel free to let me know!
Robrecht

@gavinsimpson
Copy link
Owner

gavinsimpson commented Jun 12, 2018 via email

@jarioksa
Copy link
Contributor

There are but a couple of places where the get.lam() function name needs to be changed. It seems there is more work in fixing & debugging the tag -> ord change.

@rcannood
Copy link
Author

I think the changes will indeed be fairly minimal. It's just substituting get.lam with project_to_curve, and tag with ord. If you like, I can create a pull request for analogue with what I think needs to be changed.

Are there any features you would like to see in princurve?

@rcannood
Copy link
Author

rcannood commented Aug 2, 2018

Hello @gavinsimpson and @jarioksa,

I wish to move forward with deprecating principal.curve and get.lam. Did you already have time to take a look at this? I have proposed changes in PR #21 which would make analogue work with future versions of princurve.

Kind regards,
Robrecht

@rcannood
Copy link
Author

rcannood commented Oct 8, 2018

Hello @gavinsimpson and @jarioksa,

principal.curve() and get.lam() have both been fully deprecated starting from princurve 2.1.3. Since 2.1.3 is on its way to CRAN, analogue will be generating deprecation warnings.

Robrecht

@gavinsimpson
Copy link
Owner

Thanks @rcannood I got the email from CRAN this morning about the change. I'll be dealing with this later in the week.

@gavinsimpson gavinsimpson self-assigned this Oct 9, 2018
gavinsimpson added a commit that referenced this issue Oct 9, 2018
… get.lam(), replacing it with project_to_curve() (Issue #20 )
@gavinsimpson
Copy link
Owner

This is now fixed and is on CRAN. Thanks @rcannood

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants