search

RTXteam / RTX

Software repo for Team Expander Agent (Oregon State U., Institute for Systems Biology, and Penn State U.)
https://arax.ncats.io/
MIT License
34 stars 20 forks source link

xDTD errors in CI #2257

Closed dkoslicki closed 3 months ago

dkoslicki commented 3 months ago

While all tests passed locally for me and @kvnthomas98 , it appears that xDTD tests are throwing errors on the Jenkins CI/CD. I assume this is something back-end. Assigning to @chunyuma or @kvnthomas98 depending on who has time/bandwidth.

chunyuma commented 3 months ago

@dkoslicki, I will take a look in the next few days.

chunyuma commented 3 months ago

Hi @edeutsch or @amykglen, for this issue, I found this message saying the update xdtd database not present locally. I'm wondering if we need to manually update xdtd model somewhere for CI/CD. It seems like this might be the reason for the errors on the CI/CD.

amykglen commented 3 months ago

hi @chunyuma - yes, I believe the database manager still needs to be run manually on cicd.rtx.ai once upon changes to config_dbs.json. see this part of the wiki: https://github.com/RTXteam/RTX/wiki/CI-system-(via-GitHub-actions)#the-runners-workflow

Screen Shot 2024-03-27 at 1 29 49 PM (@sundareswarpullela can correct me if I'm wrong)

chunyuma commented 3 months ago

Ah, I see. It is good to know about this. Thanks so much @amykglen. I will try seeing if it can fix this issue.

chunyuma commented 3 months ago

I close this issue because it has been fixed after updating the xdtd database on cicd.rtx.ai. Thanks @amykglen.