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.

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

Feature/galactic noise adder upgrade channel dts #591

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

sjoerd-bouma
Copy link
Collaborator

Previously, the galactic noise adder generated random galactic noise for each channel. This PR expands the selection of available galactic sky models (thanks @karenterveer) and uses consistent galactic noise between all channels (thanks @MoritzReichert), taking into account the phase differences due to the different antenna positions.

TODO:

  • fix imports of optional sky models; probably wrap them in try:... except ImportError:... clauses
  • check that the phase adjustment is correct, or at least a good approximation for antennas at different depths
  • update docstrings
  • is it correct to have random polarizations for each channel still?
  • Replace print statements with logger

karenterveer and others added 2 commits July 13, 2023 15:01
Included various sky models, moved sky noise generation out of run and into begin procedure
The channels have different positions, so there is a phase difference in the signals between the channels, which is now taken into account.
@sjoerd-bouma sjoerd-bouma mentioned this pull request May 23, 2024
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.

None yet

3 participants