Skip to content

DRAFT : Migrate to TM18-based Deregionizer#161

Open
EmyrClement wants to merge 19 commits into
p2l1pfp:L1PF_15_1_Xfrom
EmyrClement:deregionizer_tm18
Open

DRAFT : Migrate to TM18-based Deregionizer#161
EmyrClement wants to merge 19 commits into
p2l1pfp:L1PF_15_1_Xfrom
EmyrClement:deregionizer_tm18

Conversation

@EmyrClement

Copy link
Copy Markdown

PR description:

Modify the inputs to the deregionizer to represent TM18 barrel and endcap inputs + TM6 HF inputs. Also add a pattern file wirter for the inputs to the deregionizer.

Some details:

  • To handle boards with different tmux factors in the deregionizer input class, the tmux becomes parameter has become a parameter of the BoardInput struct, rather than overall deregionizer input class
  • I had to modify how the link placement info is propagated back to the DeregionizerProducer to make sure all necessary info was available to produce the pattern files, and avoid rederiving the same information just to produce the pattern files.

TODO:

  • Some of the checks for the pattern file writer may be unnecessary
  • Understand why "dummy" links are needed when specifying puppiInputLinks for the pattern file writer

PR validation:

CI in correlator-common/layer2, branches/pipelines to be added.

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.

2 participants