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.

Sign up for GitHub

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

Jump to bottom

Upgrade RcppParallel to avoids warnings from CRAN #28

Merged
merged 16 commits into from
Nov 11, 2023
Merged

Upgrade RcppParallel to avoids warnings from CRAN #28

merged 16 commits into from
Nov 11, 2023

Conversation

egpivo
Copy link
Owner

@egpivo egpivo commented Feb 11, 2023 •
edited
Loading

Objective

  • According to the latest release of RcppParallel, we upgrade the version in order to pass the CRAN's tests.

Changes

  • Upgrade RcppParallel to 5.1.7
  • Reformat code lines
  • Update the Makevars configuration
  • Correct typos
  • Add a unit test of zzz.R

Example

Coverage
image

@egpivo
Copy link
Owner Author

egpivo commented Feb 11, 2023

Wait for the CRAN update.

@egpivo egpivo self-assigned this Feb 11, 2023
@egpivo
Copy link
Owner Author

egpivo commented Nov 11, 2023 •
edited
Loading

image

Solved by correcting the path for the missing libs
image

@egpivo egpivo merged commit f643ad4 into master Nov 11, 2023
9 checks passed
@egpivo egpivo deleted the enhancement/upgrade-rcppparallel branch November 12, 2023 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@egpivo egpivo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@egpivo