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

Fcs mudst #200

Merged
merged 22 commits into from
Nov 17, 2021
Merged

Fcs mudst #200

merged 22 commits into from
Nov 17, 2021

Conversation

jdbrice
Copy link
Contributor

@jdbrice jdbrice commented Nov 17, 2021

Implements StEvent->StMuDst for FCS
Provides MuDst classes for StFcsHit, StFcsCluster, StFcsPoint and stores event level info in StFcsInfo. (Same methodology as FMS).

jdbrice and others added 22 commits June 11, 2021 10:16
update from upstream on Friday, June 11, 2021
Updated StarGeo.xml to be able to acces this in dev2021
Copy link
Member

@starsdong starsdong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The updated interfaces to MuDst look straightforward. The new StMuFcs* files are also standard for data member writing and reading.

@starsdong
Copy link
Member

Dmitri and Grigory, Daniel is the owner of MuDst code. If you can help take a review of the new additions for including FCS data into MuDst. It is kind of urgent so we can get the data into fast offline MuDst. Thanks

@plexoos
Copy link
Member

plexoos commented Nov 17, 2021

I have enough on my plate so I'll pass. Just comment here when it is ready to merge.

@starsdong starsdong merged commit ffbaf7e into star-bnl:main Nov 17, 2021
@jdbrice jdbrice deleted the fcs-mudst branch December 3, 2021 21:56
jml985 pushed a commit that referenced this pull request Dec 7, 2021
* Initial readme on how to sparse checkout StarVMC

* add .gitignore for working directories StarGeometry and xgeometry

* edit description comment

* Rotated middle disk by 30 deg to mactch actual installation.

* simplified base file for fst geometry
Updated StarGeo.xml to be able to acces this in dev2021

* Simplified FST geometry v1.1 ( cooling tubes, ribs, cooling tube holder, heatsinks (draft) )

* Adjustments to commented out simplified heatsinks and included 30 deg rotation of middle disk

* Fixed syntax issue with middle disk rotation

* Updated material definition for combined stainless and novec

* Added 3 more cooling tubes, compressed heatsinks into single volumes in PEEK bases, removed unnecessary code

* Compressed heatsinks and added 3 additional cooling tube volumes (simplified cooling tube connectors)

* accept gavins changes. removed some dummy not used lines

* Ring removed.

* remove README for pull request

* Fixes for PR

* Initial addition of Fcs to MuDst, with StEvent->MuDst and MuDst->StEvent support

* Addition of StMuFcsInfo, store reconstruction flag and detector indices, add helper methods to StMuFcsCollection

* Fix old comments for FCS classes

Co-authored-by: Flemming Videbaek <videbaks@rcas6010.rcf.bnl.gov>
Co-authored-by: Flemming Videbaek <videbaek@bnl.gov>
Co-authored-by: gavin-wilks <gavin.wilks@gmail.com>
Co-authored-by: Dmitri Smirnov <dmixsmi@gmail.com>
marrbnl pushed a commit that referenced this pull request Dec 8, 2021
* Initial readme on how to sparse checkout StarVMC

* add .gitignore for working directories StarGeometry and xgeometry

* edit description comment

* Rotated middle disk by 30 deg to mactch actual installation.

* simplified base file for fst geometry
Updated StarGeo.xml to be able to acces this in dev2021

* Simplified FST geometry v1.1 ( cooling tubes, ribs, cooling tube holder, heatsinks (draft) )

* Adjustments to commented out simplified heatsinks and included 30 deg rotation of middle disk

* Fixed syntax issue with middle disk rotation

* Updated material definition for combined stainless and novec

* Added 3 more cooling tubes, compressed heatsinks into single volumes in PEEK bases, removed unnecessary code

* Compressed heatsinks and added 3 additional cooling tube volumes (simplified cooling tube connectors)

* accept gavins changes. removed some dummy not used lines

* Ring removed.

* remove README for pull request

* Fixes for PR

* Initial addition of Fcs to MuDst, with StEvent->MuDst and MuDst->StEvent support

* Addition of StMuFcsInfo, store reconstruction flag and detector indices, add helper methods to StMuFcsCollection

* Fix old comments for FCS classes

Co-authored-by: Flemming Videbaek <videbaks@rcas6010.rcf.bnl.gov>
Co-authored-by: Flemming Videbaek <videbaek@bnl.gov>
Co-authored-by: gavin-wilks <gavin.wilks@gmail.com>
Co-authored-by: Dmitri Smirnov <dmixsmi@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.

None yet

6 participants