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

Added CMR Validation #307

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Added CMR Validation #307

wants to merge 1 commit into from

Conversation

binni979
Copy link
Contributor

@binni979 binni979 commented Dec 6, 2024

This is a PR to solve this issue.

Description:
The CMR Team recommended that pyQuARC use the CMR API for basic validation checks (metadata schema, controlled vocabulary) rather than having a separate validation. If CMR validation changes, they want to ensure that users are not using outdated validation in pyQuARC. They also want to verify consistency between what results are provided to users in pyQuARC and CMR.

Steps Implemented

  1. _validate_with_cmr function: Validates the collection using the CMR API.
  2. add_cmr_response function: Categorizes API responses as warnings or errors.
  3. Integrated these functions into main.py:
  • Validates collections via the API.
  • Appends API response errors as cmr_errors and warnings as cmr_warnings in the output of the error.

@binni979 binni979 self-assigned this Dec 6, 2024
@binni979 binni979 requested review from slesaad and xhagrg and removed request for xhagrg December 6, 2024 03:27
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.

1 participant