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

Fixing failed vignette build #170

Closed
wants to merge 5 commits into from
Closed

Fixing failed vignette build #170

wants to merge 5 commits into from

Conversation

Max-Bladen
Copy link
Collaborator

@Max-Bladen Max-Bladen commented Mar 7, 2022

SOURCE:
#169

@Max-Bladen Max-Bladen self-assigned this Mar 7, 2022
@Max-Bladen Max-Bladen changed the title First test - not letting plotLoadings run Fixing failed vignette build Mar 7, 2022
@Max-Bladen
Copy link
Collaborator Author

Max-Bladen commented Mar 7, 2022

Unable to reproduce this issue in a local R or Rmd file.

Thought issue was exclusive to plotLoadings(), but even when all chunks with this function are set to eval = FALSE, GitHub action fails at different point in the code.

Searching for length(class2) within all files yields nothing, so it is likely an issue caused by a function within one of the packages mixOmics is dependent on.

Might need your help on this one @aljabadi

@Max-Bladen
Copy link
Collaborator Author

Even when all code chunks are set to eval = FALSE within vignette.Rmd, the checks still fail. It is drawn from different functions, but the error that is raised is the same.

@aljabadi
Copy link
Collaborator

aljabadi commented Mar 7, 2022

@Max-Bladen let's see if #174 is successful. If so, we'll use the stable version of R and Bioc until the dust settles.

@aljabadi
Copy link
Collaborator

aljabadi commented Mar 7, 2022

@Max-Bladen build is now fine with stable versions. We can safely ignore these changes :)

@aljabadi aljabadi closed this Mar 7, 2022
@Max-Bladen Max-Bladen deleted the issue-169 branch December 13, 2022 22:43
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