Closed ChristopherMancuso closed 1 month ago
@falquaddoomi I removed some files like we talked about. Also, I've uploaded the new tar.gz files to the GCP bucket. Maybe make sure those files work before closing this PR?
Hey @ChristopherMancuso, thanks; I just looked over your PR and it seems reasonable.
Once I check that the new files are set up correctly I'll redeploy the GCP functions, check that they work, then approve this PR for you to merge. (Good idea to check them first, thanks for that.)
Hey @ChristopherMancuso, seems that your changes to the archives fixed the issue; after redeploying the functions I'm able to query both of them successfully. I'll go ahead and approve this, and then we can merge.
FYI, if you ever want to run the function deployment manually (e.g., if you changed the data files in the bucket, which is a change that the current deploy workflow doesn't pick up), you can follow these steps to do so:
It seemed better to have data processing scripts be at GenePlexusZoo_backend. So deleting some files in the PR.