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

Merge SpacecraftAttitudeMap into SpacecraftAttitudeExposureTable #230

Open
hiyoneda opened this issue Jul 29, 2024 · 0 comments
Open

Merge SpacecraftAttitudeMap into SpacecraftAttitudeExposureTable #230

hiyoneda opened this issue Jul 29, 2024 · 0 comments
Labels
imaging refactoring Code refactoring and API changes. spacecraftfile
Milestone
v3.0 - DC3

Comments

@hiyoneda
Copy link
Contributor

hiyoneda commented Jul 29, 2024 •
edited
Loading

SpacecraftAttitudeMap and SpacecraftAttitudeExposureTable were developed in different contexts, but they are basically the same concept: the accumulated exposure time for each scatt bin.
We can add the satellite altitude information / SC earth location to one of them for the PSR calculation with non-zenith pointing.
It seems appropriate that the merged class is a part of the SpacecraftFile module.
@israelmcmc israelmcmc moved this to Todo in cosipy Jul 29, 2024
@israelmcmc israelmcmc added spacecraftfile refactoring Code refactoring and API changes. labels Jul 29, 2024
@israelmcmc israelmcmc added this to the v3.0 - DC3 milestone Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
imaging refactoring Code refactoring and API changes. spacecraftfile
Projects
cosipy
Status: Todo
Milestone
v3.0 - DC3
Development

No branches or pull requests
2 participants
@israelmcmc @hiyoneda