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 multiple phenotypes to samples #106

Merged
merged 3 commits into from
Sep 12, 2023
Merged

✨ add multiple phenotypes to samples #106

merged 3 commits into from
Sep 12, 2023

Conversation

bunop
Copy link
Member

@bunop bunop commented Sep 12, 2023

Description

This PR will add multiple phenotypes to samples by adding a new import_multiple_phenotypes.py script

Related Issue

closes #103

Motivation and Context

This PR is required to add multiple phenotypes to the same samples

How Has This Been Tested?

Changes were tested with CI, by loading multiple phenotypes and then ensuring SMARTER-backend, r-smarter-api and SMARTER-frontend work without problems

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change which improve code itself)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@bunop bunop added the enhancement New feature or request label Sep 12, 2023
@bunop bunop added this to the SMARTER database v1.0.0 milestone Sep 12, 2023
@coveralls
Copy link

Coverage Status

coverage: 94.293% (-0.1%) from 94.419% when pulling 4ac42ad on issue-103 into ceabf8c on devel.

@bunop bunop merged commit 8712e91 into devel Sep 12, 2023
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
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants