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

fixes from CRAN #520

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

fixes from CRAN #520

wants to merge 2 commits into from

Conversation

jakob-r
Copy link
Sponsor Member

@jakob-r jakob-r commented Jul 17, 2022

Just transferring ripleys fixes/changes logged here

CRAN package akima has a restricted-use licence. Package interp has a
FOSS licence and provides a drop-in replacement for akima's functions

bicubic bicubic.grid bilinear bilinear.grid interp interp2xyz interpp

So please investigate if akima can be replaced by interp, and either
submit an updated version or let CRAN and Albrecht Gebhardt know what
the issues were, before 2022-07-04 to safely retain your package on CRAN.

Packages DiceEval mlrMBO seem not to use akima at all.

Packages
bmstdr dynaTree erpR flipr laGP plgp vegdata
do not seem to use akima conditionally, which is expected for all
suggested packages and especially for those which are not universally
available.

Also not sure why akima was needed.

@pat-s
Copy link
Member

pat-s commented Aug 2, 2022

Ahh, that's where the {interp} issue in the tests comes from. Let's see if the tests are now happy after adding {interp} so suggests and maybe we can directly remove {akima}.

@pat-s
Copy link
Member

pat-s commented Aug 29, 2022

@jakob-r Am I right to assume that these changes were already commited to mlrMBO on CRAN?

@jakob-r
Copy link
Sponsor Member Author

jakob-r commented Sep 14, 2022

Yes. As you can see in the linked github cran mirror.

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

Successfully merging this pull request may close these issues.

None yet

2 participants