Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

[MFT] Use residual instead of measurement to make Mille records#10726

Merged
shahor02 merged 4 commits intoAliceO2Group:devAliceO2Group/AliceO2:devfrom
arakotoz:pr-mille-w-resarakotoz/AliceO2:pr-mille-w-resCopy head branch name to clipboard
Feb 8, 2023
Merged

[MFT] Use residual instead of measurement to make Mille records#10726
shahor02 merged 4 commits intoAliceO2Group:devAliceO2Group/AliceO2:devfrom
arakotoz:pr-mille-w-resarakotoz/AliceO2:pr-mille-w-resCopy head branch name to clipboard

Conversation

@arakotoz
Copy link
Contributor

@arakotoz arakotoz commented Feb 7, 2023

This PR is hijacked to also bring the following modifications:

  • store and provide access to MillePede alignment corrections, errors and pulls
  • use ClassDefOverride instead of ClassDef to avoid countless warnings by ROOT interpreter when running macro calling MillePede devoted classes

@arakotoz
Copy link
Contributor Author

arakotoz commented Feb 8, 2023

Hi @robincaron13 and @shahor02,

The failed build/O2/o2 process reported the failure to be due to

475/475 Test #432: o2sim_hepmc ..........................................................................***Timeout 400.06 sec

May I ask you some advice on the possible steps to solve this reported failure? The changes in the PR are a priori not related to o2sim_hepmc. Can this failure be due to a temporary issue on the machine that ran the build? It is possible to re-run this check?

Thanks in advance for your help,

Cheers,
Andry

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 @arakotoz
this happens quite often, not related to your PR, merging.

@shahor02 shahor02 merged commit b503777 into AliceO2Group:dev Feb 8, 2023
@arakotoz arakotoz deleted the pr-mille-w-res branch February 9, 2023 10:55
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.