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
Seurat update #5339
Seurat update #5339
Conversation
remove one empty line
008fa10
to
fc18f6b
Compare
tools/seurat/macros.xml
Outdated
</requirements> | ||
</xml> | ||
<xml name="norm" label="Normalizing data" expanded="true"> | ||
<param name="low_thresholds" type="integer" value="1" label="Low threshold for filtering cells" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
min/max for integers and floats?
<filter>@FUNCTION_CITE@</filter> | ||
<filter>function['markers_file']</filter> | ||
</data> | ||
<data name="markers_cite_tabular" format="tabular" from_work_dir="markers_out.tsv" label="${tool.name} cite-seq on ${on_string}: Markers list"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tabular files should have a metadata section here to add the column names
The current version is rigid and less practical. We probably have to make it modular like the Scanpy toolkit or even a similar set of tools as Scanpy. Then adding CITE-Seq or other modalities will be easier I guess. If you need CITE-Seq urgently, we can continue with the current structure in this update. |
It has been requested that we add it asap. I don't mind future updates relatively quickly, but if we could get this feature in sooner, that would be appreciated. The rigidity of seurat is because seurat objects haven't historically been useable across other tools, so I made it a closed system. If that should be changed, I'm open to that being the step after this. |
add CITE-Seq test
fix linting and tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #5373
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Thanks @nsoranzo! |
FOR CONTRIBUTOR:
Adds cite-seq functionality to seurat, permits intermediate output files, and early stops for partial runs