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 fill scheme info in TOF event time checker #7610

Merged
merged 3 commits into from
Nov 17, 2021
Merged

Conversation

noferini
Copy link
Collaborator

No description provided.

@noferini
Copy link
Collaborator Author

noferini commented Nov 16, 2021

Sorry for all recent changes.
Now PR is frozen.

@chiarazampolli @njacazio @ercolessi
Calibrations are now loaded from ccdb at clusterization level for channel-offsets and time-slewing (via TOF api).

checked on EPN with 100 TF and everything went fine.

Just I got problem to access ccdb with an active alien token with a dpl, no problem with a macro.
Was this behavior already seen (see below)?

[3578963:TOFClusterer]: The lines below might hint at the cause of the crash.
[3578963:TOFClusterer]: You may get help by asking at the ROOT forum https://root.cern.ch/forum
[3578963:TOFClusterer]: Only if you are really convinced it is a bug in ROOT then please submit a
[3578963:TOFClusterer]: report at https://root.cern.ch/bugs Please post the ENTIRE stack trace
[3578963:TOFClusterer]: from above as an attachment in addition to anything else
[3578963:TOFClusterer]: that might help us fixing this issue.
[3578963:TOFClusterer]: ===========================================================
[3578963:TOFClusterer]: #6 0x00007f0ed0c63b31 in TJAlienCredentials::checkCertValidity(char const*) () from /home/fnoferin/alice/sw/slc8_x86-64/libjalienO2/0.1.3-local1/lib/libjalienO2.so
[3578963:TOFClusterer]: #7 0x00007f0ed0c65298 in TJAlienCredentials::selectPreferedCredentials() () from /home/fnoferin/alice/sw/slc8_x86-64/libjalienO2/0.1.3-local1/lib/libjalienO2.so
[3578963:TOFClusterer]: #8 0x00007f0ed1e39370 in o2::ccdb::CcdbApi::init (this=0x7f0ed3564040 o2::ccdb::BasicCCDBManager::instance()::inst, host=...) at /home/fnoferin/alice/sw/SOURCES/O2/dev/0/CCDB/src/CcdbApi.cxx:78

@chiarazampolli
Copy link
Collaborator

Ciao @noferini ,

Great that everything works!

Yes, there is some issue with SSL, being investigated (also QC does not work if you have a token). Please, be patient with it...

Chiara

@noferini
Copy link
Collaborator Author

Ciao @noferini ,

Great that everything works!

Yes, there is some issue with SSL, being investigated (also QC does not work if you have a token). Please, be patient with it...

Chiara

Good. Then it is not a my problem.
I have a workaround for that so I can continue my tests.
Thanks for the confirmation.

@chiarazampolli
Copy link
Collaborator

Note that the CVMFS distribution worked (at least till some days ago, then I did not try again).

@noferini
Copy link
Collaborator Author

Hi @shahor02
if you don't have any objection can you approve this PR?

Copy link
Collaborator

@shahor02 shahor02 left a comment

Choose a reason for hiding this comment

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

Hi @noferini

The PR is fine, but could you please clarify the meaning of the

static std::vector<int> mFillScheme;
? I see it is not the vector of the interacting bunches as reported by the LHC but something you compute from TOF signal. Why?

@shahor02 shahor02 merged commit 4ba1dff into AliceO2Group:dev Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants