Skip to content

Conversation

@binni979
Copy link
Contributor

@binni979 binni979 commented Nov 4, 2024

Overview
This pull request introduces a new check to verify the presence of OPeNDAP links in granule URL fields. The goal is to ensure that OPeNDAP links are included in the URL fields of granule metadata.
New Check Details
Check: Verify if OPeNDAP links are present in specified URL fields.
Fields:
Granule/OnlineResources/OnlineResource/URL (echo-g)
RelatedURLs/URL (umm-g)

Check Logic
The check inspects the specified URL fields for the presence of "opendap" in the URL.
If "opendap" is not found, a warning is generated, recommending the addition of an OPeNDAP link for this granule.

Implementation Steps
Added a new function, opendap_link_check, to perform the URL validation.
Added rule_mapping.json to include this new check.
Added checks.json to enable the OPeNDAP link check.
Added check_message.json to define warning messages for missing OPeNDAP links.

Testing
Tested Granules:
Granules that have opendap in URL, No opendap error was reported.

  1. echo-g: G1463204472-LAADS
  2. echo-g: G1479429385-LAADS
  3. echo-g: G1689034590-OB_DAAC
  4. umm-g: G2271018132-POCLOUD
  5. umm-g: G2081781961-POCLOUD

umm-g: G1997478801-LARC_ASDC (Does not contain "opendap" in URL or have URL type "OPENDAP DATA") – Error: Warning generated to recommend adding an OPeNDAP link.

Screenshot 2024-11-04 at 3 38 50 PM

@binni979 binni979 requested review from slesaad and xhagrg November 4, 2024 21:45
@binni979 binni979 self-assigned this Nov 4, 2024
@binni979 binni979 changed the title Add new check for OPeNDAP for for granules Add new check for OPeNDAP for granules Nov 5, 2024
Comment on lines 303 to 305
"data_type": "custom",
"check_function": "opendap_link_check",
"available": true
Copy link
Member

Choose a reason for hiding this comment

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

add a tab spacing before this

Copy link
Collaborator

Choose a reason for hiding this comment

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

Done.

Comment on lines 1075 to 1080
"failure": "No OPeNDAP URL is provided in the granule fields. An OPeNDAP link is recommended for data access.",
"help": {
"message": "OPeNDAP links allow for direct data access through the OPeNDAP protocol.",
"url": "https://wiki.earthdata.nasa.gov/display/CMR/Related+URLs"
},
"remediation": "Recommend providing an OPeNDAP in the granule's Online Resources or Related URLs fields for enhanced data accessibility."
Copy link
Member

Choose a reason for hiding this comment

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

this needs tab space too

Copy link
Collaborator

Choose a reason for hiding this comment

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

Resolved

@lavanya3k lavanya3k merged commit 9287b7a into dev Oct 13, 2025
1 check passed
@lavanya3k lavanya3k mentioned this pull request Oct 17, 2025
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.

5 participants