Skip to content

Code update for Run18 embedding#481

Merged
plexoos merged 2 commits intostar-bnl:SL19bfrom
zhux97:SL19b
Feb 2, 2023
Merged

Code update for Run18 embedding#481
plexoos merged 2 commits intostar-bnl:SL19bfrom
zhux97:SL19b

Conversation

@zhux97
Copy link
Copy Markdown
Contributor

@zhux97 zhux97 commented Jan 24, 2023

Patches needed for Run18 embedding are assembled in this PR. These patches are:

  • RDO map is updated and iTPC RDO is added for sector 20 in Run18.
  • TpcRS parameters for the second half of Run18 27 GeV dataset is added.
  • Offline cluster finder (RTS and StTpcRTSHitMaker) is updated with the correct gain interface.
  • A newer StTpcRSMaker is included to correct the 1st pad row.

@zhux97
Copy link
Copy Markdown
Contributor Author

zhux97 commented Jan 24, 2023

Embedding test have been performed for Run18 AuAu27 GeV, the QA plots can be found in
https://www.star.bnl.gov/protected/embedding/AutoIndex.php?dir=pr481test/

@starsdong
Copy link
Copy Markdown
Member

Xianglei, thank you very much for pushing forward. Will it be possible that you can give a (short) update on this tomorrow at the S&C meeting?

@zhux97
Copy link
Copy Markdown
Contributor Author

zhux97 commented Jan 25, 2023 via email

@starsdong
Copy link
Copy Markdown
Member

Thank you, Xianglei. That would be very helpful.

@zhux97
Copy link
Copy Markdown
Contributor Author

zhux97 commented Jan 30, 2023

Hi All, I have prepared some slides to describe the updates in this PR,
https://drupal.star.bnl.gov/STAR/system/files/embedding_status_20230130_PR481_0.pdf

@plexoos plexoos merged commit 7ad0a18 into star-bnl:SL19b Feb 2, 2023
plexoos added a commit that referenced this pull request May 10, 2023
Patches needed for Run18 Isobar (st_physics) embedding are assembled.
The patched codes are extracted from PR #466, PR #481.

---------

Co-authored-by: Dmitri Smirnov <dmixsmi@gmail.com>
Co-authored-by: Gene Van Buren <gene@bnl.gov>
Co-authored-by: Grigory Nigmatkulov <nigmatkulov@gmail.com>
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.

3 participants