biothings / smartapi-kg.js

A nodejs library for generate knowledge graphs based on SmartAPI Specifications
Apache License 2.0
0 stars 4 forks source link

Association kg #75

Open rjawesome opened 1 year ago

rjawesome commented 1 year ago

https://github.com/biothings/BioThings_Explorer_TRAPI/issues/599 Association can be specified in the yaml through association_id property, ie.

...
outputs:
      - id: NCBIGene
        semantic: Gene
biolink_association_type: TestToTest
...
rjawesome commented 1 year ago
  1. biolink: prefix is added to associations when meta_knowledge_graph endpoint is called
  2. name in x-bte yaml changed to biolink_association_type
codecov[bot] commented 1 year ago

Codecov Report

Merging #75 (867987a) into main (b426011) will increase coverage by 0.92%. The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
+ Coverage   86.18%   87.10%   +0.92%     
==========================================
  Files          30       30              
  Lines         731      737       +6     
  Branches      148      154       +6     
==========================================
+ Hits          630      642      +12     
+ Misses        100       94       -6     
  Partials        1        1              
Impacted Files Coverage Δ
..._builder/async_operations_builder_with_reasoner.ts 39.70% <0.00%> (-1.84%) :arrow_down:
src/parser/endpoint.ts 98.52% <ø> (+1.47%) :arrow_up:
...s_builder/sync_operations_builder_with_reasoner.ts 84.21% <100.00%> (+0.64%) :arrow_up:

... and 3 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

colleenXu commented 1 year ago

Noting that this may not be ready to put on dev; we may have to wait until I've finished some curation work (adding the biolink_association_type to some x-bte operations) and tested these PRs...

tokebe commented 1 year ago

Note: Requires merge conflict fixes.