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

[IBCDPE-793] Implement Great Expectations for the proteomics_tmt Dataset #119

Merged
merged 7 commits into from
Feb 9, 2024

Conversation

BWMac
Copy link
Contributor

@BWMac BWMac commented Feb 5, 2024

Description:

This PR implements a GX expectation suite for the proteomics_tmt dataset.

Please let me know if there are any additional expectations I should add, or if there are any adjustments to be made to the expectations I have already implemented.

Notes:

  • Values I have chosen for expectations like expect_column_values_to_be_between are based on looking at the example dataset on Synapse.
  • This change has been tested by running adt test_config.yaml. Example results can be downloaded here.

@BWMac BWMac marked this pull request as ready for review February 5, 2024 22:12
Copy link
Member

@thomasyu888 thomasyu888 left a comment

Choose a reason for hiding this comment

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

🔥 LGTM! same as the other PR, will wait for Jess / Jaclyn to comment on the validation rules.

Copy link
Contributor

@jaclynbeck-sage jaclynbeck-sage left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Contributor

@JessterB JessterB left a comment

Choose a reason for hiding this comment

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

A couple of potential nitpicks re. isNA checks, but otherwise lgtm.

Copy link

sonarcloud bot commented Feb 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@BWMac BWMac merged commit 55b5e70 into dev Feb 9, 2024
9 checks passed
@BWMac BWMac deleted the bwmac/IBCDPE-793/gx_proteomics_tmt branch February 9, 2024 16:23
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.

4 participants