-
Notifications
You must be signed in to change notification settings - Fork 15
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
Documentation "quick-start" #103
Comments
Thanks! Can you please share the full command you are using fir quantification? We'll fix bugs in the documentation that may have arisen from the continued development of the software. |
Hi! This the command I used, same as in previous post but with line breaks
in the doc there is this line I hope this helps. |
Ohh! I see. The problem is that we have massively upgraded In addition to the
This should then work equivalently to what is currently in the tutorial. Please let us know if this works, and great catch. Thanks for reporting it! —-Rob |
Thanks, Rob! The above parameters for the docker/singularity images worked. No more errors regarding input parameters.
Maybe I should now switch to the documentation here, https://alevin-fry.readthedocs.io/en/latest/index.html, and tutorials referenced there. Also, the docker/singularity images are not mentioned anymore in the "readthedocs" documentation. Would you then recommend to use only the versions from Many thanks! |
@DongzeHE any thoughts about this error above? It gets to the very last step. Is there any difference in where e.g. pyroe puts the t2g vs roe? |
Again, apologies for all of the difficulty here. The problem this time is that, again, the tool that build the splici transcriptome was updated, and uses a different default name for the t2g file. The proper command should be :
note the only difference from your command is that we pass
instead of
which represents the new name of the t2g file. Sorry for the confusion. Hopefully this works, and we'll update the documentation accordingly! --Rob |
Hi @rob-p Yes, now everything worked perfectly and without warnings nor errors. Many thanks for your rapid support! Best regards, |
Hi @schmucr1, Sorry for the inconvenience. As simpleaf copies and pastes the t2g files when building the index, if you don't want to check the name of the
Best, |
Thank you for the explanations. I will consider this naming conventions for the upcoming runs. On another note and not related to this issue, do you think that Many thanks and kind regards, |
Hi @schmucr1, Short answer:Yes. You need to pass the location of UMI and Cellular barcode in the technical read (read 1) as a custom geometry to simpleaf. See here for details. Long answer:Upon checking, I think alevin-fry is able to handle this dataset. However, it will be a little bit tricky because the geometry specification (the positions of UMI and cellular barcode in read1) of that protocol is unclear to me. In the paper you shared, it mentioned that they used the prep set as another study. And in that study, the authors referred to the original preprint of that prep set.
Therefore, I would suggest you ask the authors about the geometry specification of the prep set they used and set the appropriate custom geometry in simpleaf accordingly. @rob-p: Please correct me if I misunderstood what's happening. Thanks. Best, |
I agree, the right course of action, first, is to really understand the geometry. From there, figuring out how to provide it should be straightforward. One point of information is that while the mapping tools (both --Rob Update: Further inspection suggests that multiple ranges probably will work with salmon. It's only an issue parsing custom geometry with multiple ranges per piece for piscem. But, since the next release of simpleaf will be the first to support piscem, we can just fix it first ;P. |
Hello
I have some questions regarding the documentation in https://github.com/COMBINE-lab/alevin-fry#a-quick-start-run-through-on-sample-data
I read and worked through the quick-start and pull the latest singularity image,
singularity pull docker://combinelab/usefulaf:latest
There some discrepancies between that version and the documentation:
-l 91
, but there is a parameter-r
.error: Invalid value 'u' for '--forced-cells <FORCED_CELLS>': invalid digit found in string
So, I am not sure whether the documentation fits to the latest (docker/singularity) version?
Thank you and best regards,
R.
PS: and when I run with it, for example, with option
--knee
I get an unknown errorThe text was updated successfully, but these errors were encountered: