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

plotArrow Error: Aesthetics must be either length 1 or the same as the data #150

Closed
aljabadi opened this issue Sep 3, 2021 · 2 comments · Fixed by #184
Closed

plotArrow Error: Aesthetics must be either length 1 or the same as the data #150

aljabadi opened this issue Sep 3, 2021 · 2 comments · Fixed by #184
Assignees
Labels
bug Something isn't working

Comments

@aljabadi
Copy link
Collaborator

aljabadi commented Sep 3, 2021

See https://mixomics-users.discourse.group/t/error-in-plotarrow-of-splsda/689

@aljabadi aljabadi added the bug Something isn't working label Sep 3, 2021
@aljabadi aljabadi self-assigned this Sep 3, 2021
@eacton
Copy link

eacton commented Nov 18, 2021

Hi! I am also having this error with plotArrow for splsda objects. Just wondering if there have been any fixes or work-arounds yet. Thanks!

@shaman-narayanasamy
Copy link

shaman-narayanasamy commented Dec 1, 2021

Hi,

I am having an issue with this plot as well. Looking forward to the fixes.

For more information/context, I am following the MINT case study. Here are more details of the errors received when executing plotArrow(mint.splsda.res):

Error: Aesthetics must be either length 1 or the same as the data (125): x, y, xend and yend Run `rlang::last_error()` to see where the error occurred.

Executing rlang::last_error() yields the following:

<error/rlang_error>
Aesthetics must be either length 1 or the same as the data (125): x, y, xend and yend
Backtrace:
 1. (function (x, ...) ...
 2. ggplot2:::print.ggplot(x)
 4. ggplot2:::ggplot_build.ggplot(x)
 5. ggplot2:::by_layer(function(l, d) l$compute_aesthetics(d, plot))
 6. ggplot2:::f(l = layers[[i]], d = data[[i]])
 7. l$compute_aesthetics(d, plot)
 8. ggplot2:::f(..., self = self)
 9. ggplot2:::check_aesthetics(evaled, n)

@Max-Bladen Max-Bladen self-assigned this Mar 9, 2022
@Max-Bladen Max-Bladen added this to High Priority in bladen-devel-bugs Mar 9, 2022
@Max-Bladen Max-Bladen moved this from High Priority to In Progress in bladen-devel-bugs Mar 9, 2022
Max-Bladen added a commit that referenced this issue Mar 9, 2022
@Max-Bladen Max-Bladen linked a pull request Mar 9, 2022 that will close this issue
@Max-Bladen Max-Bladen moved this from In Progress to Implemented Locally in bladen-devel-bugs Mar 9, 2022
@Max-Bladen Max-Bladen moved this from Implemented Locally to PR Successful on Branch in bladen-devel-bugs Mar 13, 2022
@Max-Bladen Max-Bladen added this to PR Successful on Branch in mixOmics-development Mar 29, 2022
@Max-Bladen Max-Bladen added the ready-to-review for all PRs that are ready to be reviewed. including complex, larger commits label Sep 7, 2022
Max-Bladen added a commit that referenced this issue Dec 13, 2022
Max-Bladen added a commit that referenced this issue Dec 13, 2022
enhance: adds checks to `plotArrow()` to prevent single-omics and P-integrative methods to be visualised

The file allows for testing of the `plotArrow()` function. Two included tests check for general functionality on DIABLO objects as well as inability to function on `(mint).(s)plsda` objects
@Max-Bladen Max-Bladen removed the ready-to-review for all PRs that are ready to be reviewed. including complex, larger commits label Dec 13, 2022
@Max-Bladen Max-Bladen moved this from Ready to Review to Merged in mixOmics-development Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
bladen-devel-bugs
PR Successful on Branch
Development

Successfully merging a pull request may close this issue.

4 participants