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.

Sign up for GitHub

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

Jump to bottom

RTC-S1 ISO XML Measured Parameters Rollout #541

Merged
merged 9 commits into from
Nov 4, 2024
Merged

RTC-S1 ISO XML Measured Parameters Rollout #541

merged 9 commits into from
Nov 4, 2024

Conversation

collinss-jpl
Copy link
Collaborator

Description

  • This branch rolls out the Measured Parameters ISO XML updates to the RTC-S1 PGE

Affected Issues

Testing

  • Branch has been tested to ensure generated ISO XML lines up with version generated by last release (v2.1.1) for both standard and static layer products

Scott Collins added 9 commits October 30, 2024 11:10
Added the escape_html field to iso_metadata_measured_parameters_confi…
5ce0755 
…g_schema.yaml
Initial addition of rtc_s1_measured_parameters.yaml config
58f456d
Added NumpyEncoder utility class for use with JSON serialization to r…
13e76f9 
…ender_jinja2.py
Added usage of NumpyEncoder to json.dumps call in base_pge.augment_me…
f0e92eb 
…asured_parameters()
Updated base_pge.augment_measured_parameters() to utilize the new esc…
80e39ea 
…ape_html setting
Updated rtc_s1_pge.py to augment measured parameters with contents of…
1cccd1e 
… descriptions config
Modified ISO XML template for RTC-S1 to use for-loop to iterate over …
0379c83 
…metadata pulled from product
Added IsoMeasuredParameterDescriptions to all RTC-S1 runconfigs
00b55fa
Fixed bug in create_test_rtc_metadata_product() where ceosAnalysisRea…
23a1285 
…dyDataDocumentIdentifier was assigned incorrectly
@collinss-jpl collinss-jpl self-assigned this Oct 30, 2024
RKuttruff
RKuttruff approved these changes Oct 30, 2024
View reviewed changes
Copy link
Contributor

@RKuttruff RKuttruff left a comment

Choose a reason for hiding this comment

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

Everything looks good. Just have a couple questions.

@collinss-jpl collinss-jpl merged commit 1b7e346 into main Nov 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RKuttruff RKuttruff RKuttruff approved these changes

@JimHofman JimHofman Awaiting requested review from JimHofman

Assignees

@collinss-jpl collinss-jpl

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ISO XML Measured Parameters Update for RTC-S1
2 participants
@collinss-jpl @RKuttruff