Skip to content

Build system can now support multiple graphs #633

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

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Conversation

DnlRKorn
Copy link
Contributor

This is a functioning deploy of a version of our build software which can build different graphs using configurations

An example of how this works can be seen in

build_mokg.sh

Where

poetry run ingest download --ingest_file src/monarch_ingest/ingest_configs/mokg_downloads.txt
bash scripts/after_download.sh

and

poetry run ingest transform --ingest_file src/monarch_ingest/ingest_configs/mokg_transforms.txt

specify which datasets to ingest and which transforms to run respectively. And the command

poetry run ingest merge --kg-name mokg

specifies to merge all of the transformed data to KGX files with the name "mokg".

@DnlRKorn
Copy link
Contributor Author

DnlRKorn commented Jul 7, 2025

@kevinschaper Finally got around to fixing the comments

@DnlRKorn
Copy link
Contributor Author

@kevinschaper Can we go ahead and merge this?

Copy link
Member

@kevinschaper kevinschaper left a comment

Choose a reason for hiding this comment

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

Sorry for sitting on this forever, it would be nice to eventually split out build tooling from graph config, so that we really clearly have two configs and one build system all sitting side by side, but even beyond the mo_kg parts, this is a lot of great improvements

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.

2 participants