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

Please consider the following formatting changes to #10437 #4

Merged

Conversation

alibuild
Copy link

@alibuild alibuild commented Dec 5, 2022

This PR cannot be merged as is. You should either run clang-format yourself and update the pull request, or merge this PR in yours.
You can find the AliceO2 coding conventions at https://github.com/AliceO2Group/CodingGuidelines.

@alibuild alibuild requested a review from fmazzasc as a code owner December 5, 2022 16:56
@alibuild alibuild force-pushed the alibot-cleanup-10437 branch from 068cd48 to e6ab8ec Compare December 5, 2022 17:03
@fmazzasc fmazzasc merged commit 2ce5b59 into fmazzasc:strangeness_tracking Dec 5, 2022
@alibuild alibuild deleted the alibot-cleanup-10437 branch December 5, 2022 17:25
fmazzasc pushed a commit that referenced this pull request Aug 2, 2024
* MCH standalone alignment module

with a common part(for both MCH and MTF) of mathematics basic codes moved to a common path under O2/Detectors/ForwardAlign

Co-Authored-By: javierecc <[email protected]>

* Please consider the following formatting changes (#4)

* Removed from PR

* Fixing formatting issue

* Add missing header for filesystem

* Fix warning with dereferenced iterator

* Remove re-initialisation of magfield for trackfitter and move residual QA plots to O2Physics task

* Update comment for track selection

* Remove unused function

* Remove re-definition of magfield which is set already with CCDB finalisation

* Fixing CI build errors

---------

Co-authored-by: javierecc <[email protected]>
Co-authored-by: ALICE Builder <[email protected]>
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