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

Bins are a pain #79

Closed
damonge opened this issue Dec 19, 2019 · 0 comments · Fixed by #80
Closed

Bins are a pain #79

damonge opened this issue Dec 19, 2019 · 0 comments · Fixed by #80

Comments

@damonge
Copy link
Collaborator

damonge commented Dec 19, 2019

Pasting this from a conversation with @amaurea

Constructing bin objects is tedious and confusing. An Nside argument is needed, and setting it incorrectly leads to confusing messages about requesting too high entries during mode-coupling matrix calculation. I also wish sensible defaults were provided for the weights argument. Ideally one would be able to say: bins = Bins(lmax=5000, dl=50) or Bins(lmax=5000, fsky=0.03) or Bins([[0,50],[50,100],[150,200],...])

I agree with the above.

@damonge damonge mentioned this issue Dec 19, 2019
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 a pull request may close this issue.

1 participant