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

Synchrotron, new small scales #134

Merged
merged 11 commits into from
Nov 12, 2022
Merged

Conversation

zonca
Copy link
Member

@zonca zonca commented Nov 12, 2022

  • save kernel name to execute with papermill
  • notebook executed by @giuspugl, black-formatted
  • add more documentation to notebook

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@zonca zonca mentioned this pull request Nov 12, 2022
@zonca
Copy link
Member Author

zonca commented Nov 12, 2022

@giuspugl there was no change in the modulation of temperature, so I kept the same beta for synchrotron of the last release, please advise if you would like instead to implement the 2 slopes approach with beta as well.

@zonca
Copy link
Member Author

zonca commented Nov 12, 2022

Beta synchrotron

image
image
image

@zonca
Copy link
Member Author

zonca commented Nov 12, 2022

Test s5 vs s6 passes at NERSC both at 23 and 44 GHz

@zonca zonca marked this pull request as ready for review November 12, 2022 21:11
@zonca zonca merged commit 93116eb into galsci:main Nov 12, 2022
@giuspugl
Copy link
Contributor

giuspugl commented Nov 14, 2022

there was no change in the modulation of temperature, so I kept the same beta for synchrotron of the last release

This is correct, both temperature small scales and beta maps are left untouched, it is fine to keep beta and curvature templates as they were before. Please just make sure that the new modulation map for polarization small scales is updated .

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.

2 participants