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

Readme Peer Review #2

Open
yavyx opened this issue Oct 18, 2019 · 1 comment
Open

Readme Peer Review #2

yavyx opened this issue Oct 18, 2019 · 1 comment

Comments

@yavyx
Copy link

yavyx commented Oct 18, 2019

The Good:

  • Nice that you included a picture!
  • The diagram helps a lot with following the pipeline steps.
  • The project description is not very long but does a very good job of giving the reader a high-level understanding.

The not so Good:

  • In the diagram, it seems that TraIN is used only to cross-reference the immune PCCMs with the brain RNASeq data. In the project description, it seems that TraIN also includes the previous steps to generate the PCCMs. Could you clarify which steps are part of TraIN and which aren't?
@LuciaGuerri
Copy link
Collaborator

Hi Javier,
Thank you for your feedback. You are totally on the spot with the TraIN on the pipeline being misleading. I will definitely re-design that, as is indeed very important. Yes, the whole pipeline is TraIN! Thank you! :)
Lucia

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

No branches or pull requests

2 participants