Skip to content

Add Galaxy tutorial for breast cancer spatial transcriptomics analysis from Galaxy-EISTA - #7025

Draft
khaled196 wants to merge 25 commits into
galaxyproject:mainfrom
bioinfbrad:eista
Draft

Add Galaxy tutorial for breast cancer spatial transcriptomics analysis from Galaxy-EISTA#7025
khaled196 wants to merge 25 commits into
galaxyproject:mainfrom
bioinfbrad:eista

Conversation

@khaled196

Copy link
Copy Markdown
Contributor

Add Galaxy tutorial for breast cancer single cell analysis from Galaxy-EISTA

@khaled196 khaled196 changed the title Add Galaxy tutorial for breast cancer single cell analysis from Galaxy-EISTA Add Galaxy tutorial for breast cancer spatial transcriptomics analysis from Galaxy-EISTA Aug 4, 2026
khaled196 and others added 2 commits August 4, 2026 11:47
…utorial.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

@nilchia nilchia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @khaled196
Please also include the generated images in the training as well.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the "signals" from italic to normal.

The text in the box is outside of the box.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for changing. The genes should be italic. They are human genes, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes those are human genes. I removed them for now until find better annotating method.

Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
khaled196 and others added 3 commits August 4, 2026 15:20
…utorial.md

Co-authored-by: Amirhossein Nilchi <66441226+nilchia@users.noreply.github.com>
…utorial.md

Co-authored-by: Amirhossein Nilchi <66441226+nilchia@users.noreply.github.com>
@khaled196
khaled196 requested a review from dianichj as a code owner August 4, 2026 17:32
@khaled196

Copy link
Copy Markdown
Contributor Author

@nilchia Thank you, Amir, for your comments. Please check the tutorial again to see if I have addressed the comments. I will run it now on use.galaxy to see if I missed anything.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these two the only cell types in the data?
Can you please compare it to other papers/trainings using this data to check this?
thanks

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for changing. The genes should be italic. They are human genes, right?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From this plot, I don't think there are only 2 cell types here

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the total count should be regressed out, no?

Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
Comment thread topics/single-cell/tutorials/spatial-breast-cancer-tme-EISTA/tutorial.md Outdated
@nilchia

nilchia commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Hi @khaled196, thank you again.

I started to review again and made some comments, but now it seems to me that it is mainly AI-generated.

In this case, I cannot review and correct your AI agent; this is something you should do.

Please make this PR draft, review it yourself, and then mark it as ready for review only when you have really tested it (and also checked the rendering locally).

For example, in the md file you are saying that 3 resolutions will be tested, and in the figure that you added, there are seven resolutions.

@khaled196
khaled196 marked this pull request as draft August 5, 2026 16:25
@khaled196

Copy link
Copy Markdown
Contributor Author

Thank you, @nilchia. I apologise for not reviewing the generated text carefully before pushing it. I left the manual validation until the end, which allowed several issues to remain.

Regarding the CellTypist results, I need to investigate them further. My current understanding is that each Visium spot can contain transcripts from multiple cells, so the predicted labels should not be treated as pure cell identities or as a complete inventory of the cell types in the tissue.

I have now revised the tutorial, tested the steps manually in Galaxy, and updated the repository. We can review any remaining points during our meeting later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants