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

Combine needs a mass value for the signal process #1

Closed
adewit opened this issue Oct 12, 2015 · 4 comments
Closed

Combine needs a mass value for the signal process #1

adewit opened this issue Oct 12, 2015 · 4 comments

Comments

@adewit
Copy link
Collaborator

adewit commented Oct 12, 2015

For some limit calculation methods, combine needs the signal process to have a mass value specified in the shape file. CH currently lets you make a datacard + shape file without mass value, if this is done it would be nice to assign the signal process a dummy mass value so that the datacard can be used with all combine methods.

@ajgilbert
Copy link
Collaborator

Just to check I understand the issue (also reported by @thomas-mueller): the problem is purely on the CH side, in that it requires signal processes to have a valid numeric mass value to write a datacard? This then has to be passed to text2workspace with -m. What we want is a way to write signal processes without the $MASS pattern. Is that correct or is there another problem?

@adewit
Copy link
Collaborator Author

adewit commented Oct 14, 2015

Ah yes, it is only that - so yes, we want a way to write signal processes without the $MASS pattern.

@ajgilbert
Copy link
Collaborator

@adewit @ffrensch @thomas-mueller
i've tried to fix this on the ch-oct-15 branch - let me know if it works ok. now if you set a signal process mass to "*" it shouldn't put $MASS in the mapping pattern.

@adewit
Copy link
Collaborator Author

adewit commented Oct 15, 2015

Great! This works fine :-)

ajgilbert added a commit that referenced this issue May 12, 2016
Added wrapper for AddWorkspace and ExtractPdfs
mverzett pushed a commit to mverzett/CombineHarvester that referenced this issue May 30, 2017
Width interpolation updates and standard flow
perrozzi added a commit that referenced this issue Jun 28, 2018
mflechl pushed a commit that referenced this issue Dec 3, 2018
danielwinterbottom pushed a commit to danielwinterbottom/CombineHarvester that referenced this issue Apr 9, 2019
…18-dev

fixing physics model. removeing stuff not needed. using degrees for a…
lucastorterotot referenced this issue in lucastorterotot/CombineHarvester Dec 6, 2019
andreypz pushed a commit to andreypz/CombineHarvester that referenced this issue Jul 2, 2020
Adding an option to have JEC uncertanties split into sources
danielwinterbottom pushed a commit to danielwinterbottom/CombineHarvester that referenced this issue Sep 3, 2020
tahuang1991 added a commit to tahuang1991/CombineHarvester that referenced this issue Oct 15, 2020
acarvalh pushed a commit to acarvalh/CombineHarvester that referenced this issue Feb 1, 2021
lauridsj pushed a commit to lauridsj/CombineHarvester that referenced this issue Jul 13, 2023
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

No branches or pull requests

2 participants