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

Clean up keyword n_avg #197

Open
rlbyrne opened this issue Feb 21, 2020 · 3 comments
Open

Clean up keyword n_avg #197

rlbyrne opened this issue Feb 21, 2020 · 3 comments
Labels
documentation A request for more documentation

Comments

@rlbyrne
Copy link
Contributor

rlbyrne commented Feb 21, 2020

The keyword n_avg need a default reported in the dictionary. It seems to be used in both gridding and Healpix outputs and has different defaults in different parts of the code. We need to clean up, document, and possibly rename the keyword.

@bhazelton
Copy link
Member

Agreed. I find it particularly confusing that there is another keyword, nfreq_avg which means something totally different but sounds very similar every time I see it.

@mwilensky768
Copy link

@bhazelton not that it solves this issue, but I'm working on a PR that just changes nfreq_avg to beam_nfreq_avg which is slightly more illustrative.

@bhazelton
Copy link
Member

That will help a lot, thanks!

@nicholebarry nicholebarry added the documentation A request for more documentation label Jan 25, 2021
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

No branches or pull requests

4 participants