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

Fix additions and add GHA R CMD CHECK #90

Merged
merged 4 commits into from
Nov 20, 2021
Merged

Conversation

andrjohns
Copy link
Collaborator

Duplicate of #88, but with the base branch coming from stan-dev/rstantools rather than my fork, to allow the Github Actions to run

* test fixes and add GHA tests

* test reorder

* test reorder

* test reorder

* test reorder

* Tidy
@jgabry
Copy link
Member

jgabry commented Oct 7, 2021

Awesome, thanks! I can confirm that I don't get the error on my computer when I run things locally but it's great to have GHA checks. Thanks for setting that up.

@jgabry jgabry linked an issue Oct 7, 2021 that may be closed by this pull request
@andrjohns
Copy link
Collaborator Author

Hmm for some reason the tests are hanging on the Windows run, but I can't reproduce locally. Will see if I can find a fix

@jgabry
Copy link
Member

jgabry commented Oct 7, 2021

Hmm for some reason the tests are hanging on the Windows run, but I can't reproduce locally. Will see if I can find a fix

Yeah that's strange. Thanks for working on this

@andrjohns
Copy link
Collaborator Author

It looks like it's hanging specifically at the final linking stage when creating and compiling a package, but only for Windows:

   C:/rtools40/mingw64/bin/g++ -shared -s -static-libgcc -o rstanlm.dll tmp.def RcppExports.o stanExports_lm.o -LD:/a/_temp/Library/RcppParallel/lib/x64 -ltbb -ltbbmalloc -LD:/a/_temp/Library/RcppParallel/lib/x64 -Wl,-rpath,D:/a/_temp/Library/RcppParallel/lib/x64 -ltbb -ltbbmalloc -LC:/R/bin/x64 -lR

Very weird.

@jgabry
Copy link
Member

jgabry commented Oct 8, 2021

It looks like it's hanging specifically at the final linking stage when creating and compiling a package, but only for Windows:

   C:/rtools40/mingw64/bin/g++ -shared -s -static-libgcc -o rstanlm.dll tmp.def RcppExports.o stanExports_lm.o -LD:/a/_temp/Library/RcppParallel/lib/x64 -ltbb -ltbbmalloc -LD:/a/_temp/Library/RcppParallel/lib/x64 -Wl,-rpath,D:/a/_temp/Library/RcppParallel/lib/x64 -ltbb -ltbbmalloc -LC:/R/bin/x64 -lR

Very weird.

@andrjohns So this is just happening on Windows with GHA but not on your local Windows system?

@bgoodri Any ideas what's going wrong here?

@hsbadr
Copy link
Member

hsbadr commented Oct 8, 2021

Very weird.

This looks like a GHA issue, not related to rstantools or this PR. Try to change the version of the actions (e.g., r-lib/actions/setup-r@master instead of r-lib/actions/setup-r@v1) or test the workflow at a different time (sometimes GHA for R fails in the morning or at night).

@andrjohns
Copy link
Collaborator Author

I wasn't able to figure out why it was hanging on the windows GHA, so have just removed for now. The new actions will test on Ubuntu & MacOS, which is still more than the previous Travis tests (just linux).

@andrjohns
Copy link
Collaborator Author

@jgabry Would this be all good to go? Once it's out to CRAN I can submit patches to some packages for 2.26+ compatibility

Copy link
Member

@hsbadr hsbadr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to merge this to fix the master branch for rstan >= 2.26.

@andrjohns andrjohns merged commit 623fb94 into master Nov 20, 2021
@jgabry jgabry deleted the rstantools-226-error branch March 10, 2022 18:25
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.

switch to github actions for CI
3 participants