Skip to content

Guannan's tune up for 7p7GeV_2021, https://www.star.bnl.gov/~xgn1992/…#505

Merged
starsdong merged 2 commits intostar-bnl:mainfrom
fisyak:TFG.Export.031823
Mar 22, 2023
Merged

Guannan's tune up for 7p7GeV_2021, https://www.star.bnl.gov/~xgn1992/…#505
starsdong merged 2 commits intostar-bnl:mainfrom
fisyak:TFG.Export.031823

Conversation

@fisyak
Copy link
Copy Markdown
Member

@fisyak fisyak commented Mar 19, 2023

…star/Tpc/2021/7p7GeV_2021/TFG23a_revisit8/index.html, mv TF1F to StarRoot

@starsdong
Copy link
Copy Markdown
Member

Thanks, Yuri and Guannan. I see the change is done for the TpcResponseSimulator.y2008.C, does this mean it applies to all simulations after y2008?

@fisyak
Copy link
Copy Markdown
Member Author

fisyak commented Mar 20, 2023

Hi Xin, TpcResponseSimulator.y2008.C is applied to run 8 only. The run 8 is the first run where we had ALTRO cheap. But for this run ALTRO did not have any tail cancellation parameters. The first run with tail cancellation was run 9.

…20 16:58:03 to 2022-07-28 23:59:59, remove starClockOnl.devE.C
@fisyak fisyak requested a review from dmarkh as a code owner March 21, 2023 21:11
@starsdong starsdong requested a review from zhux97 March 21, 2023 21:24
@starsdong
Copy link
Copy Markdown
Member

Thank you, Irakli for reviewing this. Thank you, Yuri for the update of cleaning up of the tpcRDOMask tables from previous commit (PR#500). This PR looks to be urgently needed for both 7.7 GeV embedding and library built for upcoming OO production. Jason, Dmitry (also Xianglei), if you can help review it and comment/approve this PR soon. Thanks

Copy link
Copy Markdown
Contributor

@dmarkh dmarkh left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@starsdong
Copy link
Copy Markdown
Member

Thank you all for confirming and thank Yuri for taking extra care of the clean-up of the StarDb tables. I think this PR is ready to be merged.

@starsdong starsdong merged commit 44e89aa into star-bnl:main Mar 22, 2023
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.

5 participants