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.

Sign up for GitHub

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

Jump to bottom

try out: use individual automats, remove use of drug approvals #52

Merged
merged 1 commit into from
Nov 1, 2024
Merged

try out: use individual automats, remove use of drug approvals #52

merged 1 commit into from
Nov 1, 2024

Conversation

colleenXu
Copy link
Contributor

@colleenXu colleenXu commented Oct 31, 2024 •
edited
Loading

Copied from Slack:

Here's my analysis of how the "new TRAPI KP additions" are affecting BTE's automated test harness performance.
My recommendations: try adjusting BTE-CI's API_LIST #52

  • Try using the old set of individual Automat APIs
  • Remove use of Multiomics Drug Approvals
    See how it does in this weekend's automated test run. If the tests I identified pass + number of TopAnswer passes is improved, then consider putting it into the hotfix CI->Test deployment.
    Caveat: I didn't have time to dig into the biology/content of the answers.

@colleenXu colleenXu changed the title feat: use individual automats, remove use of drug approvals try out: use individual automats, remove use of drug approvals Oct 31, 2024
@tokebe tokebe merged commit ec5841d into main Nov 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@colleenXu @tokebe