-
Notifications
You must be signed in to change notification settings - Fork 218
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
Adding Squidpy #1318
base: master
Are you sure you want to change the base?
Adding Squidpy #1318
Conversation
@astrovsky01 Tagging for squidpy |
@tcollins2011 Thanks for initiating squidpy! 🚀 It would be nice to implement each of the scverse packages similarly to the IUC implementation of Scanpy, i.e., group some related functions into a single Galaxy tool of that package. If we have a collection of tools for squidpy functions, then we can also use them for other spatial data like merfish or nanostring. Moreover, the scverse is currently developing a new datatype called SpatialData for spatial omics. Maybe it will completely replace anndata for spatial analysis. In that case, it will become hard to adapt such tools. |
I would agree that it would be useful to separate Squidpy in a similar manner to Scanpy. Right now this first pass at squidpy is only compatible with 10x visium data, however the tool can take many more inputs and we haven't fully implemented all of its end functionality. I'll continue to try and and separate the tool out as time goes on, but I'll probably need to find a way to tackle the persistent paths necessary for the image container object to work correctly within Squidpy. |
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.
please add a .shed.yml ... and probably a suite one so we can add more tools into this directory.
-l 'None' | ||
#end if | ||
-s spatial/tissue_hires_image.png | ||
-x "${prefix}" |
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.
single quotes
@@ -0,0 +1,154 @@ | |||
<tool id="squidpy" name="squidpy" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" python_template_version="3.5" profile="21.05"> |
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.
<tool id="squidpy" name="squidpy" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" python_template_version="3.5" profile="21.05"> | |
<tool id="squidpy" name="squidpy" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" profile="21.05"> |
]]></command> | ||
<inputs> | ||
<!-- CHANGE FORMAT LATER --> | ||
<param argument="--visium" type="data" format="visium.tar.gz" label="Visium dataset tarball"/> |
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.
A user probably does not know what a tarball is. Just Visum dataset might be ok.
Implementation of squidpy spatialomics for 10x visium data analysis.