Skip to content

Conversation

@rararani
Copy link
Contributor

@rararani rararani commented May 22, 2022

Implemented the notebook for calculating Swath Width as a function of focal length.

Things to note:

  1. the skew angle's default value did not change. What changed is that skew_angle is 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.
  2. the field-of-view vector requires slit dimensions. An arbitrary slit size was used just for demonstration purposes of the pipeline. These values will likely need to be changed
  3. Swath width's default units are km, while spatial resolution's units are in m. With the current slit size and skew angles, the swath width's values (when converted to metres) are far larger than that of the spatial resolution's values, which results in a very flat plot for spatial resolution (kinda becomes meaningless to visualize). This is why I chose to plot the swath width and the spatial resolution on different graphs. I can combine them if needed

@rararani rararani requested a review from DM1122 May 22, 2022 23:10
@rararani rararani self-assigned this May 22, 2022
Copy link
Member

@DM1122 DM1122 left a 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.

@rararani rararani requested a review from DM1122 May 23, 2022 06:14
@DM1122 DM1122 merged commit 8f0ba27 into dev May 23, 2022
@DM1122 DM1122 deleted the swath-width branch May 23, 2022 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants