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

Add PPI demo #1065

Closed
dondi opened this issue Nov 14, 2023 · 5 comments
Closed

Add PPI demo #1065

dondi opened this issue Nov 14, 2023 · 5 comments

Comments

@dondi
Copy link
Owner

dondi commented Nov 14, 2023

For parity with GRNs, we need to add PPI demos to the app. @kdahlquist will generate the file and this can be added to the app in the same way that the existing demos are added.

@dondi
Copy link
Owner Author

dondi commented Nov 21, 2023

We’ll start with an export based on the database then move on to Dr. Dabir’s data when ready

@kdahlquist
Copy link
Collaborator

Demo #5 PPI Network (Yeastmine - SGD 2023-03-29; 18 proteins, 81 edges).xlsx

I created this network from the database. It is all the proteins that interact with Aim32p, which is the protein that Dr. Dabir studies.

I exported it from GRNsight and then modified the filename.

It has only a "network" sheet. I changed the text in cell A1 to read "protein 1/protein 2".

It loads back into GRNsight OK, but currently, it loads as a GRN, not PPI because there's nothing to tell GRNsight that it is a PPI network.

In doing this, I think I uncovered a bug, but will report in a different issue.

@kdahlquist
Copy link
Collaborator

I'm open to shortening the filename to something like:

Demo #5 PPI (18 proteins, 81 edges)

"Network" is probably redundant. If we shorten it this way, we lose the provenance of the data, but that could be put into the documentation. This could be better due to readability in the dropdown menu.

@dondi
Copy link
Owner Author

dondi commented Nov 28, 2023

File is ready to go—we will use the content of cell A1 to determine the Network Mode to switch to, as noted in #1066. Label the demo as noted in the comment above

Demos are encoded directly as JSON objects; the easiest way to convert is probably to temporarily add a console.log to the code which displays the converted network object. When opening the file, this network JSON will appear in the Developer Console and can then be transcribed to the source code in demo-workbooks.js

As noted, we will use cell A1 of the Network sheet to determine the Network Mode. Further, we will need to adjust the warning to adapt to these changes

@dondi dondi assigned ntran18 and unassigned kdahlquist Nov 28, 2023
dondi added a commit that referenced this issue Dec 5, 2023
dondi added a commit that referenced this issue Dec 5, 2023
@kdahlquist
Copy link
Collaborator

Verified present in beta 7.0.8

This was referenced Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants