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

ARAX is failing TRAPI 1.5 validation in CI #2275

Closed maximusunc closed 1 month ago

maximusunc commented 2 months ago

Given this query: https://ars.ci.transltr.io/ars/api/messages/0cc6b8e1-1814-48b7-b461-98a06574681e

Auxiliary Graphs are required to have an empty list of attributes and not null. https://github.com/NCATSTranslator/ReasonerAPI/blob/master/TranslatorReasonerAPI.yaml#L800

saramsey commented 2 months ago

Putting this on the agenda for today's all-hands meeting.

edeutsch commented 2 months ago

@saramsey What was the outcome from the AHM? I don't see outcomes in the agenda.

We have another opportunity to push to TEST in the next couple days.

Would it be possible to have someone on this issue as a high prio so that we can get this fixed in TEST in the next push?

dkoslicki commented 2 months ago

@kvnthomas98 would you be able to address this like you did with #2272? Apparently this is very high priority

kvnthomas98 commented 2 months ago

Yes Sure! working on it now

saramsey commented 2 months ago

Thank you @kvnthomas98

kvnthomas98 commented 1 month ago

Hi @maximusunc , This is fixed in CI now

"aux_graph_N1_99": {
        "attributes": [],
        "edges": [
          "N1_99"
        ]
      }
maximusunc commented 1 month ago

Confirmed in CI that ARAX is passing TRAPI validation. Thanks everyone!!