Skip to content

Apply pointing correction to SAT sim#1675

Merged
ykyohei merged 2 commits into
masterfrom
apply_pointing_sim
Jun 24, 2026
Merged

Apply pointing correction to SAT sim#1675
ykyohei merged 2 commits into
masterfrom
apply_pointing_sim

Conversation

@ykyohei

@ykyohei ykyohei commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Apply both boreight pointing model and pointing wobble to SAT simulation. wobble is optional.
Another PR of bb-awg-scripts: simonsobs/bb-awg-scripts#21

Tested this using boresight pointing of obs_1731438172_satp3_1111111, ws1, f090
The middle one is difference of no wobble and wobble input only. We see residual in small scale as expected.
The bottom one is difference of no wobble and wobble correction both input and output (mapmaking). SInce these two scans slgihtly different region of sky, we see residual at the edge of map, as expected.

map diff1 diff2

logger.info("Reading in simulated map")
demod_mm.from_map(aman, sim_map, wrap=True, modulated=True, interpol=interpol)
demod_mm.from_map(aman, sim_map, wrap=True, modulated=True,
interpol=interpol, sight=sight)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just for curiosity, does this also need to be propagated when we do the inverse operation, i.e. tod->map ? In which case we'll have to modify the small function in the filtering script

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Exactly! Inverse tod->map operation is PRed here simonsobs/bb-awg-scripts#21

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thanks for making this!
One question, which might related to Adri's question, is that is this bb-awg-scripts using map making? Wobble correction is already in here, and I though we have used this to make an atmic map.

@ykyohei ykyohei Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah I was confused as well. The data pipeline which you linked had wobble option already, but the tf pipeline is different and it is in awg repo and it didn't have wobble correction option, so I made a PR.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see, bb-pwg-scripts is used for TF. That makes sense. Thank you!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We might want to merge the two if possible (in the future) so that we don't duplicate code and use a single mapmaking function.

@ykyohei ykyohei Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The reason I didn't directly integrate wobble to mapmaking function is because it only works when all the dets belong to single wafer band.

Also I think it might be cleaner to integraye wobble to pointing model, not pointing matrix.

Comment thread sotodlib/preprocess/preprocess_util.py Outdated
@yoshinori-0778 yoshinori-0778 self-requested a review June 24, 2026 02:05
@ykyohei ykyohei merged commit 474ce41 into master Jun 24, 2026
5 checks passed
@ykyohei ykyohei deleted the apply_pointing_sim branch June 24, 2026 11:07
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.

3 participants