add functions for making detector-centered coordinate system.#1167
Open
yoshinori-0778 wants to merge 4 commits intomasterfrom
Open
add functions for making detector-centered coordinate system.#1167yoshinori-0778 wants to merge 4 commits intomasterfrom
yoshinori-0778 wants to merge 4 commits intomasterfrom
Conversation
Contributor
Author
|
@mhasself Could you review this PR? Once this is merged, I plan to work to implement the unified planet mapmaker that I curretly developped in pwg-script into sotodlib. |
bb32d72 to
720a316
Compare
Contributor
Author
|
@mhasself I am also working on improving unified map maker in pwg-script. Once the improvement get settles, I would like to make a PR for them. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds functions for making maps in detector-centered coordinate system that is useful for beam characterization.
I confirmed
get_each_detcen_Pandcalc_detcen_coadd_mapswoking properly with Jupiter observation (after pre-process applied).Current version need to execute
get_each_detcen_Pevery single detector and then coadded them.In addition, prior to
get_each_detcen_Pplanet az/el needs to be calculated.It took roughly 3min for 148 detectors with map size of 2deg and map resolution of 2arcmin.