Skip to content

fixes from CRAN#520

Draft
jakob-r wants to merge 2 commits into
mainfrom
ripley_cran_fix
Draft

fixes from CRAN#520
jakob-r wants to merge 2 commits into
mainfrom
ripley_cran_fix

Conversation

@jakob-r

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

Copy link
Copy Markdown
Member

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

pat-s commented Aug 2, 2022

Copy link
Copy Markdown
Member

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

pat-s commented Aug 29, 2022

Copy link
Copy Markdown
Member

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

@jakob-r

jakob-r commented Sep 14, 2022

Copy link
Copy Markdown
Member Author

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.

2 participants