Skip to content

Association kg#75

Open
rjawesome wants to merge 6 commits intomainfrom
association-kg
Open

Association kg#75
rjawesome wants to merge 6 commits intomainfrom
association-kg

Conversation

@rjawesome
Copy link
Contributor

@rjawesome rjawesome commented Apr 12, 2023

biothings/biothings_explorer#599
Association can be specified in the yaml through association_id property, ie.

...
outputs:
      - id: NCBIGene
        semantic: Gene
biolink_association_type: TestToTest
...

@rjawesome
Copy link
Contributor Author

  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
Copy link

codecov bot commented Apr 12, 2023

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%) ⬇️
src/parser/endpoint.ts 98.52% <ø> (+1.47%) ⬆️
...s_builder/sync_operations_builder_with_reasoner.ts 84.21% <100.00%> (+0.64%) ⬆️

... and 3 files with indirect coverage changes

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

@colleenXu
Copy link
Contributor

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
Copy link
Member

tokebe commented Oct 25, 2023

Note: Requires merge conflict fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants