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

Update docs, other changes for cookiecutter usage #133

Merged
merged 30 commits into from
May 13, 2024
Merged

Update docs, other changes for cookiecutter usage #133

merged 30 commits into from
May 13, 2024

Conversation

glass-ships
Copy link
Collaborator

@glass-ships glass-ships commented Apr 19, 2024

Closes #132 and #134

This is a fairly large one.
Several updates and changes, some needed in general, some to make it play more nicely with cookiecutter for ingests

  • Updated documentation
  • Remove TODO comments after converting them into issues in github
  • Remove some vestigial code sections
  • Add dynamic versioning
  • Add min_node_count and min_edge_count to primary source config
  • Add a check after transform that output contains at least the minimum specified number of rows, if included in config.yaml
  • Update biolink model imports
  • Update dependencies
  • Rename cli_runner.py to cli_utils.py to be less confusingly named
  • Plus some formatting noise

@glass-ships glass-ships self-assigned this Apr 23, 2024
@glass-ships glass-ships merged commit 7563b8f into main May 13, 2024
4 checks passed
@glass-ships glass-ships deleted the develop branch May 13, 2024 20:34
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.

Make necessary changes for modular ingests
2 participants