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

Review PPI database-loading script #1072

Closed
dondi opened this issue Nov 21, 2023 · 4 comments
Closed

Review PPI database-loading script #1072

dondi opened this issue Nov 21, 2023 · 4 comments

Comments

@dondi
Copy link
Owner

dondi commented Nov 21, 2023

@ntran18 has run into some issues with the PPI loading code/documentation in beta (script doesn’t complete; tables do not get populated). We should compare this to what @nchun2 is seeing and in general troubleshoot what might be going on

This may include temporarily (or permanently) revising the scripts to produce better feedback while running so that the user can take note of any progress/issues

@ntran18
Copy link
Collaborator

ntran18 commented Nov 28, 2023

Actually generate_protein_network.py would not populate any data into the database tables. It will create 5 different files that will later be used to populate data. Therefore, the tables don't contain any data.

After that, python3 loader.py | psql postgresql://localhost/postgres this script actually populates data into the database. It might stop because of duplicate protein Aad6p (might want to bold 1 from Aad16p to have a better distinguish between Aad6p and Aad16p.

After fixing the duplicate protein, if we run the script again, there might be some data already in the table. Thus, having more errors on duplicate keys. My solution was to add Truncate tables to remove existing data, and populate data again.

ntran18 added a commit that referenced this issue 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
@dondi
Copy link
Owner Author

dondi commented Jan 9, 2024

We need to look at the current deployed beta to make sure things are functional. @ntran18 and @nchun2 can re-run the scripts to test functionality on a pre-existing database; @ceciliazaragoza and @akaiap will be running these scripts for the for the first time as a part of #1078 and can also report back on functionality

@kdahlquist
Copy link
Collaborator

@ceciliazaragoza was able to get the PPI database running OK. But @akaiap wasn't. She'll see @dondi in office hours to troubleshoot, so we'll keep this open for a little while longer.

@dondi
Copy link
Owner Author

dondi commented Jan 23, 2024

@akaiap and @dondi worked on this last Thursday and got the issues resolved

@dondi dondi closed this as completed Jan 23, 2024
ntran18 added a commit that referenced this issue Jan 23, 2024
dondi added a commit that referenced this issue Feb 6, 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

6 participants