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

Updated profile LUT and call for merged cluster prob. #3380

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

pjwinnetou
Copy link
Contributor

Updated profile LUT and call for merged cluster prob. variable + usage of vertex (if exist) for cluster position correction in the profile module

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work for users)
  • Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

TODOs (if applicable)

Links to other PRs in macros and calibration repositories (if applicable)

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 35c10f9dd08e0ce0382bc4b91b1c75507993866b:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 32c8bf648457b89d930e3601237c318d603b36b0:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@pinkenburg
Copy link
Contributor

It's likely easier for you to use branches for development rather than the master branch.
the hardcoded filenames in RawClusterLikelihoodProfile.h which point to your user dir need to go, changing this PR to draft so it doesn't get merged until this is done

@pinkenburg pinkenburg marked this pull request as draft January 29, 2025 17:21
@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 2a72980e3449b1fd5232e51369fdcf87851058c2:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 7c0ba3e0ad2fe13ec87bb6918dd3694cbc3bc3c2:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 669daaa4f05dce215c795f2d029a248de0c7a6a8:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 950e39b340b28c21d6befbad2ed9519fa927db18:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 2a3c327045586ae8b4cb5ce31659752fdc1f8349:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 6d10ddd5f14210e548d2c898ef1a48a826cdcbf1:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@pinkenburg pinkenburg marked this pull request as ready for review February 12, 2025 15:49
@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit a8c7570344a6f4da125cbf323934a4df6ca89a78:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

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