Skip to content

Removed unused libraries, and added logic for missing ion parameter file in topsStack#702

Merged
rtburns-jpl merged 7 commits intoisce-framework:mainfrom
bjmarfito:fixes
Apr 26, 2023
Merged

Removed unused libraries, and added logic for missing ion parameter file in topsStack#702
rtburns-jpl merged 7 commits intoisce-framework:mainfrom
bjmarfito:fixes

Conversation

@bjmarfito
Copy link
Contributor

@bjmarfito bjmarfito commented Apr 19, 2023

  • Removed unused and duplicate library imports in mdx.py and unwrapComponents.py
  • Added check if the ion parameter file actually exists for ionospheric correction in stackSentinel.py

@bjmarfito bjmarfito changed the title Removed unused libraries and added logic for missing ion parameter file in topsStack Removed unused libraries, added python3 header, and added logic for missing ion parameter file in topsStack Apr 21, 2023
@bjmarfito bjmarfito changed the title Removed unused libraries, added python3 header, and added logic for missing ion parameter file in topsStack Removed unused libraries, and added logic for missing ion parameter file in topsStack Apr 25, 2023
@bjmarfito bjmarfito requested a review from rtburns-jpl April 26, 2023 02:35
Copy link
Member

@rtburns-jpl rtburns-jpl left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@rtburns-jpl rtburns-jpl merged commit e77d207 into isce-framework:main Apr 26, 2023
@bjmarfito bjmarfito deleted the fixes branch April 28, 2023 04:50
yuankailiu added a commit to yuankailiu/isce2 that referenced this pull request May 23, 2023
+ resolve the conflicts since stackSentinel.py has been updated (in isce-framework#505, isce-framework#702)
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.

2 participants