Skip to content

feat(QCDDAT): CVT ML Sub-package - #1338

Draft
c-dilks wants to merge 13 commits into
developmentfrom
CVT_QCDDAT_redux
Draft

feat(QCDDAT): CVT ML Sub-package#1338
c-dilks wants to merge 13 commits into
developmentfrom
CVT_QCDDAT_redux

Conversation

@c-dilks

Copy link
Copy Markdown
Member

PR opened on behalf of @zieglerv

Redux of CVT_QCDDAT branch, since

  • keep @zieglerv's original branch as is
  • branch urwt_detector-change-hook is based off of CVT_QCDDAT

@c-dilks

Copy link
Copy Markdown
MemberAuthor

Hmm.. Look's like Veronique's commits are authored by ziegler (which GitHub thinks is an unknown user), not @zieglerv; anyway, it's clear these are both Veronique, but we should make sure @zieglerv is attributed when we merge this (otherwise it may erroneously look like I did all this work, when all I've done so far was rebase!)

@zieglerv

Copy link
Copy Markdown
Collaborator

Sorry I committed them from my personal Mac that is why the author name is different

@baltzell

Copy link
Copy Markdown
Collaborator

See #1339 to address the ReconstructionEngine issues.

@c-dilksc-dilks mentioned this pull request Jul 9, 2026
@c-dilksc-dilks changed the title feat: CVT QCDDAT ML Sub-packagefeat(QCDDAT): CVT ML Sub-packageJul 27, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@c-dilks@zieglerv@baltzell