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

javascript error in console leading to perpetual wait for search results in alignment pipeline #420

Open
adf-ncgr opened this issue Apr 6, 2022 · 4 comments
Assignees
Labels

Comments

@adf-ncgr
Copy link
Contributor

adf-ncgr commented Apr 6, 2022

A seemingly innocuous request:
https://gcv.legumeinfo.org/gene;lis=cicar.CDCFrontier.gnm1.ann1.Ca_21954?algorithm=repeat&match=10&mismatch=-1&gap=-1&score=30&threshold=25&bmatched=20&bintermediate=10&bmask=10&linkage=average&cthreshold=20&neighbors=50&matched=4&intermediate=5&sources=lis&bregexp=&border=chromosome&regexp=&order=distance
leads to console error:
main.c67d416a58e79492.js:1 ERROR RangeError: Invalid array length
and somewhat misleading pipeline status:
image
which leaves the user in some perplexity.

@alancleary alancleary self-assigned this May 10, 2022
@alancleary alancleary added the bug label May 10, 2022
@alancleary
Copy link
Contributor

Thanks for bumping this issue. It got buried among the other issues! Unless it's a high priority, this probably won't get fixed until June when I'm back from vacation. It's worth noting that switching the alignment algorithm to Smith-Waterman prevents the error: https://gcv.legumeinfo.org/gene;lis=medtr.A17.gnm5.ann1_6.MtrunA17Chr3g0078741?q=medtr.A17.gnm5.MtrunA17Chr3:1000000-1500000&sources=lis&algorithm=smith-waterman&match=10&mismatch=-1&gap=-1&score=30&threshold=25&bmatched=20&bintermediate=10&bmask=10&linkage=average&cthreshold=20&neighbors=70&matched=4&intermediate=5&bregexp=&border=chromosome&regexp=&order=distance

@adf-ncgr
Copy link
Contributor Author

thanks, not urgent I just happened to run into again when doing something else. Good to know it is only an issue for the repeat algorithm, I guess that should help narrow down the cause later.

@alancleary
Copy link
Contributor

Good to know it is only an issue for the repeat algorithm, I guess that should help narrow down the cause later.

Yeah, that's not at all surprising. Stitching together all the forward and reverse repeat alignment fragments is overly complicated at the moment. Hopefully addressing this issue helps simplify the code!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants