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

"subscript out of bounds" #72

Open
dmaher14 opened this issue Sep 22, 2021 · 6 comments
Open

"subscript out of bounds" #72

dmaher14 opened this issue Sep 22, 2021 · 6 comments

Comments

@dmaher14
Copy link

I'm trying to run through the sample code outlined here. Everything works fine up until the generateReportSA() function, which produces the following error:

Quitting from lines 57-77 (SangerAlignment_Report.Rmd)
Error in params$SangerAlignment@contigList[[1]] : subscript out of bounds

Any idea what might be going wrong here?

@Kuanhao-Chao
Copy link
Collaborator

Kuanhao-Chao commented Oct 5, 2021

Hi @dmaher14 ,

Sorry for my late reply. The document on Bioconductor is the old version. You can check the latest example on Github homepage Link or sangeranalyseR documentation.

There are some examples you can just copy and paste to rerun everything. Please let me know if you still have any problems.

Howard

@dmaher14
Copy link
Author

dmaher14 commented Oct 6, 2021

Hi @dmaher14 ,

Sorry for my late reply. The document on Bioconductor is the old version. You can check the latest example on Github homepage Link or sangeranalyseR documentation.

There are some examples you can just copy and paste to rerun everything. Please let me know if you still have any problems.

Howard

Hi Howard, thank for replying. The documentation you linked me to is actually what I tried first. I'm still a little confused as to which version is "current." After loading the sangeranalyseR library, the main function arguments don't appear to match that documentation. For example, the SangerAlignment function no longer seems to accept the argument REGEX_SuffixForward but instead requires suffixForwardRegExp. Is this possibly the source of the error?

@Kuanhao-Chao
Copy link
Collaborator

Kuanhao-Chao commented Oct 7, 2021

Hi @dmaher14 ,

Please install sangeranlyseR again from github link. After installing it, it should work! Please let me know if it doesn't work.

Best,
Kuan-Hao

@dmaher14
Copy link
Author

dmaher14 commented Oct 9, 2021

Hi @dmaher14 ,

Please install sangeranlyseR again from github link. After installing it, it should work! Please let me know if it doesn't work.

Best, Kuan-Hao

Hi Kuan-Hao,

After messing with the program a bit more I am wondering if maybe some of the problems I am running into are stemming from the fact that I am trying to align multiple F and R traces together (as opposed to just one of each) to create a contig. Does sangeranalyseR allow for this?

@nick-youngblut
Copy link

I'm getting a similar error:

Error in params$forwardReadFN[[i]] : subscript out of bounds

I'm using sangeranalyseR 1.8.0. It appears that the code is making assumptions about the size of the params$forwardReadFN list, and these assumptions can break for poor quality Sanger reads.

@Kuanhao-Chao
Copy link
Collaborator

sangeranalyseR allows for aligning multiple forward (F) and reverse (R) traces together. Forward reads are stored in params$forwardReadFN[[i]], while reverse reads are stored in params$reverseReadFN[[i]].

Thanks to @nick-youngblut and @dmaher14 for reporting this bug. It should be fixed in the latest version 1.10.0.

Kuan-Hao

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

No branches or pull requests

3 participants