Skip to content

Introduce the possibility to perform spin scans#20

Merged
algebrato merged 26 commits intomasterfrom
spin_tod
Jul 21, 2021
Merged

Introduce the possibility to perform spin scans#20
algebrato merged 26 commits intomasterfrom
spin_tod

Conversation

@algebrato
Copy link
Member

@algebrato algebrato commented Dec 23, 2020

  • Modify sim_tod to introduce a check on the az lims.

  • Create an end-to-end simulations in the cookbook directory

  • Create unit-test: after a sideral day the telescope has to point on the same pixel

In order to perform spin scans.
WARNING: A spin scan cannot be uniform. Too many hits at the edges
of the sky slab. The spin velocity has to be modulated accordingly
with the distance from the north pole.
@algebrato algebrato self-assigned this Dec 23, 2020
@algebrato algebrato added the enhancement New feature or request label Dec 23, 2020
algebrato and others added 25 commits December 24, 2020 09:34
Typo in init function
update the single detector script
Now, the `atm_absorption_coefficient` uses the correct opacity
where the dry-air, O2 and H2O are taken into account, instead
the only H2O contribute.
 Changes to be committed:
	modified:   src/libcal/src/AATM_fun.cpp
Now, the sping scan is modulated as a function of the AZ angle.
In this way the pointing are well distributed on the sky and are not
still concentrated on the two slab of sky at the edges of the observed sky

The improvements are significant, firt of all, the denoise algorithm
works a lot of order o magnitude better!
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@algebrato
Copy link
Member Author

Now the simulate_spin_scan function, works and produces uniform hit_maps.

@algebrato algebrato merged commit cfd9b0f into master Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants