-
Notifications
You must be signed in to change notification settings - Fork 1
Swath width #82
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
Swath width #82
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job! I wasn't able to provide feedback on the jupyter notebook itself since its really hard to figure out what's going on when github displays all the boilerplate associated with .ipynb.
Regarding your point three, we'd really like to be plotting both spatial res and swath on the same plot to make it easier to do a visual analysis. What we can do is plot swath in km instead of converting to m. That should solve the problem of the vanishing spatial res line.
Co-authored-by: David Maranto <[email protected]>
Co-authored-by: David Maranto <[email protected]>
Co-authored-by: David Maranto <[email protected]>
Co-authored-by: David Maranto <[email protected]>
Co-authored-by: David Maranto <[email protected]>
Co-authored-by: David Maranto <[email protected]>
Implemented the notebook for calculating Swath Width as a function of focal length.
Things to note:
skew_angleis now a 2D vector instead of a scalar. B/c of this, spatial resolution now uses the skew angle in the across-track direction (i.e.skew_angle[0]). If this should be changed let me know.