Skip to content

Introducing calibration factors for each channel#2

Open
sgiardie wants to merge 10 commits intomasterfrom
new_cal
Open

Introducing calibration factors for each channel#2
sgiardie wants to merge 10 commits intomasterfrom
new_cal

Conversation

@sgiardie
Copy link
Copy Markdown
Collaborator

I have modified the function _get_calibrated_spectra inserting also calibration factors for each frequency cal_93, cal_145, cal_225. In order to preserve the previous version of the code, I added these lines to fix the new factors to one if they are not given as inputs. There was no need to modify the syslibrary class.
@paganol @mgerbino what do you think? Do you prefer having a different function?

@sgiardie
Copy link
Copy Markdown
Collaborator Author

I also merged the simonsobs/master to this branch, leaving the modifications I've done in theoryforge_mflike. Everything else should be as in simonsobs/master

@sgiardie
Copy link
Copy Markdown
Collaborator Author

I actually removed the condition to fix the new calibration params to the default if they are not present in the .yaml file used for the run, since the default should be taken from MFLike.yaml. I also added the new cal params in the self.expected_params_nuis. So if the params are not considered by the user, the treatment is the same as before

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