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

Pathways vis #26

Merged
merged 17 commits into from
Aug 31, 2023
Merged

Pathways vis #26

merged 17 commits into from
Aug 31, 2023

Conversation

Ales-ibt
Copy link
Contributor

Pahtways visualization notebook after per peer review by @SandyRogers and @KeteSakharova. We still need to improve the KEGGRESt queries that takes too long to run.

@Ales-ibt Ales-ibt requested a review from SandyRogers May 22, 2023 12:14
Copy link
Contributor Author

@Ales-ibt Ales-ibt left a comment

Choose a reason for hiding this comment

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

Looks like some changes were made by me, so I cannot approve the whole PR. I'll ask Martin to review and merge

@Ales-ibt
Copy link
Contributor Author

Ales-ibt commented Jul 4, 2023

I can see Sandy is the second reviewer, I think he will be able to approve and merge :D

@SandyRogers
Copy link
Member

@all-contributors please add @amartyanambiar for code, examples, ideas

@allcontributors
Copy link
Contributor

@SandyRogers

I've put up a pull request to add @amartyanambiar! 🎉

@SandyRogers SandyRogers merged commit 23cf412 into main Aug 31, 2023
0 of 2 checks passed
@SandyRogers SandyRogers deleted the pathways_vis branch August 31, 2023 15:46
SHillman836 pushed a commit to SHillman836/notebooks that referenced this pull request Jul 10, 2024

Co-authored-by: amartyanambiar <amartyanambiar@gmail.com>
Co-authored-by: Amartya Nambiar <51471924+amartyanambiar@users.noreply.github.com>
Co-authored-by: sandyr <sandyr@ebi.ac.uk>
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.

None yet

3 participants