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.

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

Add malacards to emc pipeline #722

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add malacards to emc pipeline #722

wants to merge 1 commit into from

Conversation

matentzn
Copy link
Member

Adds MalaCards (by GeneCards) as EMC to Mondo Ingest.

Overview

This PR:

  • One
  • Two
  • Three

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

  • Yes
  • No, updates to the docs were not necessary after careful consideration

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after
docker pull obolibrary/odkfull:dev), and no errors occurred?

  • Yes
  • No, there are no functional (code-related) changes to the pipeline in the PR, so no re-run is necessary

New Packages

Were any new Python packages added?

Were any other non-Python packages added?

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

  • Yes

  • Await further instructions from malacard team before proceeding

@matentzn matentzn self-assigned this Dec 16, 2024
@joeflack4 joeflack4 added the enhancement New feature or request label Dec 16, 2024
@@ -751,6 +752,16 @@ $(EXTERNAL_CONTENT_DIR)/mondo-otar-subset.robot.tsv:
$(EXTERNAL_CONTENT_DIR)/mondo-medgen.robot.tsv:
wget "https://github.com/monarch-initiative/medgen/releases/latest/download/medgen-xrefs.robot.template.tsv" -O $@

###### MalaCards #########
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool. Just looking at this preliminarily. See if you can re-request or tag me again to review when it is undrafted and ready.

I like their website and how the entry pages are put together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants