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

Adding reagent validations from Koenig lab, McMaster University. #157

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

zivy
Copy link
Member

@zivy zivy commented Jul 2, 2024

No description provided.

Copy link

@KoenigJFE KoenigJFE left a comment

Choose a reason for hiding this comment

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

All looks accurate to me.

Copy link
Contributor

Choose a reason for hiding this comment

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

The image for alpha smooth muscle actin is in the small intestine and linked to lymph node entry. It should be linked to a lymph node entry in the csv file.

Copy link
Contributor

Choose a reason for hiding this comment

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

This entry is correct. B220 antibody in LN. Image corresponds to entry.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good to me. Image is associated with correct file.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Contributor

Choose a reason for hiding this comment

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

The CD200 image is associated with CD11c reagent entry? This is wrong.

Copy link
Contributor

Choose a reason for hiding this comment

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

@KoenigJFE Beautiful image Josh! The caption says, "Mouse (immunized) lymph node: CD45.1 (yellow, catalog number 110718) and CD138 (magenta, catalog number 142526)". I think the yellow is CD138 (plasmablasts/plasma cells emerging from GC) and CD45.1 is magenta and you are looking in a CD45.1 mouse, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

The image associated with this entry is for CD3 and the entry is for CD81. This is wrong.

Copy link
Contributor

@radtkea radtkea left a comment

Choose a reason for hiding this comment

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

There are a few errors. Wrong images with wrong entries, I think. Figure legend as well. See comments below. Thank you for all your great work @KoenigJFE and @zivy.

@zivy
Copy link
Member Author

zivy commented Jul 8, 2024

@radtkea, the PR has been updated based on your guidance. Please re-review.

Copy link
Contributor

@radtkea radtkea left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@zivy zivy merged commit 24f8c9f into IBEXImagingCommunity:main Jul 8, 2024
3 checks passed
@zivy zivy deleted the addingReagents branch July 8, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants