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

dictionary updates #320

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

dictionary updates #320

wants to merge 2 commits into from

Conversation

bhazelton
Copy link
Member

I'd like to get these in, we need them! @dstorer and @PyxieLouStar maybe you can expand the description on no_frequency_flagging? It's a little terse at the moment.

@bhazelton bhazelton added the documentation A request for more documentation label Mar 13, 2024
Adding entry for fill_model_visibilities
@nicholebarry nicholebarry self-assigned this Aug 14, 2024
Copy link
Contributor

@nicholebarry nicholebarry left a comment

Choose a reason for hiding this comment

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

fill_model_visibilities is hardcoded to 1 in the model generation code. Do we want this to be changed on input?

Default for beam_nfreq_avg is 1 within the codebase. Default for beam_nfreq_avg is 16 within the eor_wrapper_defaults, which is perhaps were the confusion came from here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation A request for more documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants