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

add option for bandshift_dict #18

Merged
merged 2 commits into from
May 5, 2023
Merged

add option for bandshift_dict #18

merged 2 commits into from
May 5, 2023

Conversation

thibautlouis
Copy link
Collaborator

No description provided.

@thibautlouis thibautlouis requested a review from xgarrido May 5, 2023 08:21
fg_components,
fg_params,
fg_norm=None,
band_shift_dict=None):
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think band_shift_params is a better name

@@ -246,6 +251,8 @@ def get_foreground_dict(ell, external_bandpass, fg_components, fg_params, fg_nor

fg_norm: dict
the foreground normalisation. By default, {"nu_0": 150.0, "ell_0": 3000, "T_CMB": 2.725}
band_shift: dict
Copy link
Collaborator

Choose a reason for hiding this comment

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

then here it is band_shift_dict

@codecov-commenter
Copy link

Codecov Report

Patch coverage: 50.00% and project coverage change: -0.74 ⚠️

Comparison is base (fae787b) 16.14% compared to head (be2e8d5) 15.40%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
- Coverage   16.14%   15.40%   -0.74%     
==========================================
  Files          11       11              
  Lines        1220     1220              
==========================================
- Hits          197      188       -9     
- Misses       1023     1032       +9     
Impacted Files Coverage Δ
pspipe_utils/best_fits.py 41.12% <50.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@thibautlouis thibautlouis merged commit 96bd95b into main May 5, 2023
12 checks passed
@thibautlouis thibautlouis deleted the band_shift_dict branch May 5, 2023 11:12
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