Resolve "Broken pi0 ecal barrel analysis script"

Closes #47 (closed)

added bug label

marked this merge request as draft

added 1 commit

• fb06c2ad - Fixed range

Compare with previous version

added 1 commit

• 8cc79943 - testing new structure

Compare with previous version

added 1 commit

• d44b2f04 - Done?

Compare with previous version

marked this merge request as ready

Hi @whit, the pipeline is succeeding now, can I merge it?

How are you triggering the pipeline? Is it through the MR "piplines" tab?

@mariakzurek and @chao, the upstream pipeline is also broken: reconstruction_benchmarks#47 (closed)

That should get fixed first then this can be merged.

added 27 commits

• 310224b3 - 1 commit from branch master
• 42ffebcc - Checking CI
• af48e2c5 - Checking CI 2, fix error in root file
• 399d9ec4 - Checking CI 3, commented out everything after fsam
• ed6afd5b - Checking CI 4, commented out everything after sigmaOverE
• 093c3908 - Checking CI 4, commented out everything energy resolution print statement
• b263dc7d - Checking CI 5, same commenting out, changed hde_copy to h1 & moved write test to after pass/fail
• 8f183681 - saving before common bench
• 4862018c - Testing CI 6, deleting gaus function before last fit
• c6fc169c - Testing CI 7, commenting out fsam
• 52f272ca - Testing CI 8, deleting h1 and using h2 later
• 3d86cc2f - Testing CI 9, last uncomment
• d6d34144 - Done
• 32002c4d - Done 2
• 22aef288 - Done 3
• b4600795 - Done 4, didnt change anything, but calibrate failed last time
• db680894 - Done 5
• 7d1167eb - Done 6
• 198d07cb - Commented out last histogram
• 9f0fa667 - Testing fit function
• 30913319 - Hopefully done
• 11fde5ce - Moved E_rel
• 3b993cbb - new loop
• 72aa21f3 - new loop 2
• 9c6bb931 - Fixed range
• 12da6de1 - testing new structure
• 7e2d93cc - Done?

Compare with previous version

added 1 commit

• e90235a6 - Check

Compare with previous version

added 1 commit

• b87d63ce - Check 2

Compare with previous version

added 1 commit

• 8f6ae3d2 - Check 3

Compare with previous version

added 1 commit

• 7112e67f - Check 4

Compare with previous version

added 1 commit

• 4067a183 - changed h1 to h2

Compare with previous version

added 1 commit

• b6e3a321 - split loop in two

Compare with previous version

added 1 commit

• 0fd32b82 - commented out benchmark

Compare with previous version

added 1 commit

• d88db3d2 - commented out fsam

Compare with previous version

added 35 commits

• 7c99cf7b - 1 commit from branch master
• 0d9fe485 - Checking CI
• 7b394a14 - Checking CI 2, fix error in root file
• 3ed63912 - Checking CI 3, commented out everything after fsam
• 9c27f6ef - Checking CI 4, commented out everything after sigmaOverE
• d636e857 - Checking CI 4, commented out everything energy resolution print statement
• 3726e2c6 - Checking CI 5, same commenting out, changed hde_copy to h1 & moved write test to after pass/fail
• 668f39eb - saving before common bench
• 99a92d08 - Testing CI 6, deleting gaus function before last fit
• 295fe421 - Testing CI 7, commenting out fsam
• 15a4e1a7 - Testing CI 8, deleting h1 and using h2 later
• ce4c3c50 - Testing CI 9, last uncomment
• 8622cf32 - Done
• 9a7c9686 - Done 2
• e1546af5 - Done 3
• a663236b - Done 4, didnt change anything, but calibrate failed last time
• 1b5a959d - Done 5
• 02c1a746 - Done 6
• a454a6d8 - Commented out last histogram
• ef86aebe - Testing fit function
• e9a5a985 - Hopefully done
• 78f2e4bd - Moved E_rel
• aab201ba - new loop
• 1d559cea - new loop 2
• 2453e41f - Fixed range
• 657a418a - testing new structure
• 140028e8 - Done?
• 92cb3d79 - Check
• c44e97bf - Check 2
• c04f59bb - Check 3
• 4bf3696d - Check 4
• 75f570dc - changed h1 to h2
• 9b9df348 - split loop in two
• a72d8934 - commented out benchmark
• 13c911d4 - commented out fsam

Compare with previous version

added 1 commit

• 5e0d6aac - Testing both

Compare with previous version

added 1 commit

• 44bc9b4b - Trying to not draw the second fit function

Compare with previous version

added 1 commit

• 0cff0890 - Done??

Compare with previous version

marked this merge request as draft

marked this merge request as ready

added 1 commit

• 69cf47f1 - Finishing up

Compare with previous version

added 39 commits

• 3a27c116 - 1 commit from branch master
• 824c744f - Checking CI
• 23836a3f - Checking CI 2, fix error in root file
• f20bc487 - Checking CI 3, commented out everything after fsam
• 0490e5c9 - Checking CI 4, commented out everything after sigmaOverE
• 5140398c - Checking CI 4, commented out everything energy resolution print statement
• 58d737d5 - Checking CI 5, same commenting out, changed hde_copy to h1 & moved write test to after pass/fail
• ba743dfc - saving before common bench
• cb7a15b9 - Testing CI 6, deleting gaus function before last fit
• c0db9657 - Testing CI 7, commenting out fsam
• ee846b4c - Testing CI 8, deleting h1 and using h2 later
• 685881fb - Testing CI 9, last uncomment
• 694e5f4e - Done
• 5d0a4d34 - Done 2
• b3fc04fb - Done 3
• 1e8fc352 - Done 4, didnt change anything, but calibrate failed last time
• 14107966 - Done 5
• ecf74f54 - Done 6
• c91037f6 - Commented out last histogram
• 4b8c7172 - Testing fit function
• 5d2fc0cb - Hopefully done
• 6e92d222 - Moved E_rel
• 6d496fbb - new loop
• 73f5d1d1 - new loop 2
• 0bbe7301 - Fixed range
• 93e6cd1b - testing new structure
• c5c30710 - Done?
• e131f04d - Check
• 2fa779d9 - Check 2
• 21a0819a - Check 3
• 85c945c6 - Check 4
• b38dac58 - changed h1 to h2
• b8a50e06 - split loop in two
• b251b9d1 - commented out benchmark
• 8cc1c56c - commented out fsam
• 5058a8a2 - Testing both
• 6b71cd7e - Trying to not draw the second fit function
• 0ce27c9b - Done??
• 96277715 - Finishing up

Compare with previous version

merged