Skip to content

RGH recoil (trk and tof)#1312

Merged
baltzell merged 8 commits into
developmentfrom
dev_rgh
Jun 16, 2026
Merged

RGH recoil (trk and tof)#1312
baltzell merged 8 commits into
developmentfrom
dev_rgh

Conversation

@silvianic

Copy link
Copy Markdown
Collaborator

Modification of directory structure as advocated by Raffaella, addressed PR 921 and PR504. Work done by Nilanga and Silvia.

@baltzell

Copy link
Copy Markdown
Collaborator

We need to run an autoformatter/indenter on this.

@baltzell

Copy link
Copy Markdown
Collaborator

See #1313

@raffaelladevita raffaelladevita left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

there is no engine for RTOF

Comment thread etc/bankdefs/hipo4/rtof.json
@niwgit

niwgit commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

there is no engine for RTOF

I made a commit to my local branch dev_rgh on this but couldn't push to the dev_rgh branch on remote.. Looks like a permission issue.

@baltzell

Copy link
Copy Markdown
Collaborator

there is no engine for RTOF

I made a commit to my local branch dev_rgh on this but couldn't push to the dev_rgh branch on remote.. Looks like a permission issue.

You should have write access now.

@niwgit

niwgit commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

You should have write access now.

Thanks

@baltzell

Copy link
Copy Markdown
Collaborator

See #1316

@baltzell baltzell enabled auto-merge (squash) June 16, 2026 20:22
@baltzell baltzell merged commit 243ba98 into development Jun 16, 2026
22 checks passed
@baltzell

Copy link
Copy Markdown
Collaborator

Are the pull requests #504 and #921 superseded by this one, or are they still relevant?

@niwgit

niwgit commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Are the pull requests #504 and #921 superseded by this one, or are they still relevant?

PR #504 and #921 are superseded by this one. But I would keep those PRs accessible just in case someone wants to see the commit history for the two detectors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants