Skip to content

publish scores to hf code#2097

Closed
pascalwhoop wants to merge 3 commits intomainfrom
feat/matrix-score-publishing
Closed

publish scores to hf code#2097
pascalwhoop wants to merge 3 commits intomainfrom
feat/matrix-score-publishing

Conversation

@pascalwhoop
Copy link
Copy Markdown
Contributor

No description provided.

@pascalwhoop pascalwhoop requested a review from a team as a code owner February 26, 2026 18:44
@pascalwhoop pascalwhoop requested review from JacquesVergine and piotrkan and removed request for JacquesVergine February 26, 2026 18:44
Copy link
Copy Markdown
Collaborator

@matentzn matentzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great - I want to note that after some back and forth with @JacquesVergine we had decided to decouple the drug and disease list release pipelines from the KG one after all (they are now published in HF by the normal drug / disease list release process), because we wanted the KG one to be a bit more deliberate due to its size.

You are coupling the scores here to the KG - maybe a separate pipeline could be sensible, but in any case...

THIS IS AWESOME! THANKS!

@pascalwhoop
Copy link
Copy Markdown
Contributor Author

A pipeline is just a grouping of things. kedro run --nodes A,B,C --pipeline X let's you control which node to run.

So IMO we don't need 4 pipelines for 4 single nodes, just bundle them all in a pipeline and if you want to run just one node, run that node

Copy link
Copy Markdown
Member

@piotrkan piotrkan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

…lishing

# Conflicts:
#	pipelines/matrix/conf/cloud/data_publication/catalog.yml
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has had no activity in the last 14 days.

If this PR is still relevant:

  • Please update it to the latest main/master branch
  • Address any existing review comments
  • Leave a comment to keep it open

Otherwise, it will be closed in 2 weeks if no further activity occurs.

@github-actions github-actions Bot added the stale label Mar 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has had no activity in the last 14 days.

If this PR is still relevant:

  • Please update it to the latest main/master branch
  • Address any existing review comments
  • Leave a comment to keep it open

Otherwise, it will be closed in 2 weeks if no further activity occurs.

@github-actions github-actions Bot added the stale label Apr 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically closed because it has been stale for 90 days with no activity.

If you believe this is still relevant, please feel free to reopen it or create a new PR.

@github-actions github-actions Bot closed this Apr 24, 2026
auto-merge was automatically disabled April 24, 2026 00:29

Pull request was closed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants