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

DSWx PGE Integration #17

Merged
merged 6 commits into from
Nov 18, 2021
Merged

Conversation

collinss-jpl
Copy link
Collaborator

This PR adds an initial version of the `dswx_pge.py' module, which defines the tailored inheritors of the Base PGE classes/mixins to support integration with the DSWx-HLS SAS interface.

The DSWx PGE inherits all the functionality of the base PGE and adds additional validation checks for both the inputs and outputs defined by the RunConfig. Additionally, the Yamale schema for the DSWx-HLS SAS has also been added, according to its definition from the SAS User's Guide.

Lastly, this PR modifies pge_main.py to add a mapping of PGE names to classes to instantiate so the appropriate PGE to instantiate and run may be determined at run-time according to the provided RunConfig.

Scott Collins added 6 commits November 10, 2021 12:45
These attributes can then be overwritten by inheritors to ensure the correct class name is used for console or log messages
Providing a value of True for this option instructs the PgeExecutor class to invoke the SAS command line via system shell.
This provides some additional flexibility when defining dummy command-line executables for use with PGE testing.
…moved some unused codes related to "Header" processing left over from NISAR
… to the appropriate PGE class to instantiate
@collinss-jpl collinss-jpl self-assigned this Nov 10, 2021
@collinss-jpl collinss-jpl merged commit c2cf8b7 into main Nov 18, 2021
@collinss-jpl collinss-jpl deleted the 16_dswx_hls_interface_integration branch January 12, 2022 22:20
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