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

Error in plotMDS (edgeR_miRBase.r) #187

Closed
luissian opened this issue Sep 16, 2022 · 2 comments
Closed

Error in plotMDS (edgeR_miRBase.r) #187

luissian opened this issue Sep 16, 2022 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@luissian
Copy link

Description of the bug

When script is checking condition to make the MDS plot, it is checking about the number of sample.

 # Make MDS plot (only perform with 3 or more samples)
    if (length(filelist[[1]]) > 2){
        pdf(paste(header,"_edgeR_MDS_plot.pdf",sep=""))
        MDSdata <- plotMDS(dataNorm)
        dev.off()

This works in many cases but I have a situation where the length is 3 but when dataNorm is set

# TMM
    dataNorm <- calcNormFactors(dataDGE)

I get that there are only 2 columns.
I need to figure out what is wrong with the sample, but however checking should be done on the numbers of columns on dataNorm and not by the filelist

Command used and terminal output

Output error


Error in plotMDS.default(y, top = top, labels = labels, pch = pch, cex = cex,  : 
  Only 2 columns of data: need at least 3
Calls: plotMDS -> plotMDS.DGEList -> plotMDS -> plotMDS.default
Execution halted


### Relevant files

_No response_

### System information

_No response_
@luissian luissian added the bug Something isn't working label Sep 16, 2022
@apeltzer apeltzer added this to the Release 2.1.1 Patch milestone Oct 11, 2022
@apeltzer apeltzer self-assigned this Feb 17, 2023
@apeltzer
Copy link
Member

Good spot, will make a PR for a fix for this very soon!

@apeltzer apeltzer removed this from the Release 2.2.0 milestone Sep 1, 2023
@apeltzer apeltzer added this to the 2.4.0 milestone Aug 8, 2024
@apeltzer apeltzer added this to smrnaseq Aug 8, 2024
@atrigila atrigila self-assigned this Aug 16, 2024
@atrigila atrigila mentioned this issue Aug 19, 2024
11 tasks
@atrigila
Copy link
Contributor

Closed via #380

nschcolnicov pushed a commit that referenced this issue Oct 10, 2024
fix: closure bug when per flow cell manifest is not included
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
Status: Done
Development

No branches or pull requests

3 participants