-
Notifications
You must be signed in to change notification settings - Fork 38
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
load.bety.sh doesn't populate trait_covariate_associations table #403
Comments
@robkooper Can you prioritize this? It's a nuisance to always have to copy this table manually. |
Need to have #422 merged first, most of the code is there already in dump.bety.sh but commented out. |
Done. But I haven't deployed the code yet. |
@dlebauer Pull request #422 only adds the ids to the table—it's only a preliminary step toward fixing the problem. I think now @robkooper has to uncomment the commented out code he mentioned in his previous comment in order for things to work. |
@dlebauer @robkooper is there a work around until this is automated? I am running into this issue and may be why I can not use the Bulk Uploader on our instance of BETYdb (modex.bnl.gov/bety) |
@serbinsh workaround to populate the trait_covariate_associations table is to run this: |
OK, so updating the table using commands like:
And then trying bulk upload again worked for me so it seems the solution is to populate that table? |
Actually this is how I ran the command, leaving here for posterity |
It creates it but it's empty. From a developer's standpoint, it's inconvenient to have to populate this table manually. And for users doing synchonization, I don't see any reason why this table should be treated any differently from the others.
The text was updated successfully, but these errors were encountered: