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.

Sign up for GitHub

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

Jump to bottom

MFT: new MC Track Task #2115

Merged
merged 5 commits into from
Feb 15, 2024
Merged

MFT: new MC Track Task #2115

merged 5 commits into from
Feb 15, 2024

Conversation

Edingrast
Copy link
Contributor

No description provided.

@Barthelemy
Copy link
Collaborator

I think that you have a typo QCMFTTrackMCTask -> QcMFTTrackMCTask that breaks the checks.

@Edingrast
Copy link
Contributor Author

I think that you have a typo QCMFTTrackMCTask -> QcMFTTrackMCTask that breaks the checks.

Hi Barth,
Thank you for your comment. The issue has been fixed.
Sara

@diana0x0f
Copy link
Collaborator

Hi @Barthelemy ,
one of the checks (o2-cs8) is still not successful and it is not clear to us what could be the cause. Could you please have a look? Thanks!

@Barthelemy
Copy link
Collaborator

this is a spurrious error, i will merge your commit.

@Barthelemy Barthelemy merged commit aef03da into AliceO2Group:master Feb 15, 2024
6 of 7 checks passed
@benedikt-voelkel benedikt-voelkel mentioned this pull request Feb 27, 2024
Merged
Barthelemy pushed a commit that referenced this pull request Feb 28, 2024
@Edingrast @Barthelemy
MFT: new MC Track Task (#2115)
9562cee 
* MFT: new MC track task

* MFT: adding LinkDef.h and CMakeLists.txt

* MFT:modification of header file

* MFT: header file clean up, pT resolution -> 1/pT resolution, binning adjustments

* MFT: correction QCMFTTrackMCTask->QcMFTTrackMCTask in CmakeList.txt
@Edingrast Edingrast deleted the MFTMCTrackTask branch March 10, 2024 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@diana0x0f diana0x0f diana0x0f approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Edingrast @Barthelemy @diana0x0f