Skip to content

Add sar:bandwidth #26#27

Merged
m-mohr merged 1 commit into
mainfrom
bandwidth
Jul 16, 2025
Merged

Add sar:bandwidth #26#27
m-mohr merged 1 commit into
mainfrom
bandwidth

Conversation

@m-mohr
Copy link
Copy Markdown
Contributor

@m-mohr m-mohr commented Jul 15, 2025

Closes #26

@m-mohr m-mohr requested a review from emmanuelmathot July 15, 2025 16:12
@m-mohr m-mohr mentioned this pull request Jul 15, 2025
@m-mohr m-mohr requested a review from Copilot July 16, 2025 09:27
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new sar:bandwidth field to the SAR STAC extension to describe the range bandwidth of SAR instruments. The change includes schema updates, documentation, and example data to support this new field.

  • Adds sar:bandwidth as a required field in the JSON schema with validation constraints
  • Updates documentation to describe the new field and clarifies the sar:center_frequency description
  • Provides example values in Sentinel-1 sample files and updates the changelog

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
json-schema/schema.json Adds sar:bandwidth as required field with number type and minimum validation, plus validation for sar:center_frequency
examples/sentinel-1-slc-iw.json Adds example sar:bandwidth and sar:center_frequency values for Sentinel-1 SLC data
examples/sentinel-1-grd-ew.json Adds example sar:bandwidth value for Sentinel-1 GRD data
README.md Documents the new sar:bandwidth field and clarifies sar:center_frequency description
CHANGELOG.md Records the addition of sar:bandwidth and validation changes

Comment thread json-schema/schema.json
Comment thread README.md
@m-mohr m-mohr merged commit 6390fe7 into main Jul 16, 2025
2 checks passed
@m-mohr m-mohr deleted the bandwidth branch July 16, 2025 09:37
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.

Add sar:bandwidth?

3 participants