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

Setting column size factor manually #266

Closed
OlivierBBB opened this issue Oct 14, 2024 · 3 comments · Fixed by #274
Closed

Setting column size factor manually #266

OlivierBBB opened this issue Oct 14, 2024 · 3 comments · Fixed by #274
Labels
enhancement New feature or request

Comments

@OlivierBBB
Copy link
Collaborator

Certain helper columns in the HUB only exist in the "interleaved and permuted domain" but arise neither as the row permutation of other columns nor as the interleaving of existing columns from the HUB. We ought to be able to declare them with a size parameter (here: 4.)

Currently when attempting to declare them we can't tell corset that they ought to be considered as columns 4x as long as standard HUB module columns. We get the following error:

image
@OlivierBBB OlivierBBB added the enhancement New feature or request label Oct 14, 2024
@DavePearce
Copy link
Collaborator

DavePearce commented Oct 15, 2024

with a size parameter

This is referred to in corset as the length mulitplier. So you want to be able to declare a column with length multiplier that is not 1? Presumably so that it can be used in constraints involving interleaved columns, etc?

@DavePearce
Copy link
Collaborator

DavePearce commented Oct 15, 2024

Options for syntax:

(defcolumns (X :length 2))

or

(defcolumns (X :height 2))

@DavePearce
Copy link
Collaborator

DavePearce commented Oct 15, 2024

Minimal example to illustrate:

(defcolumns X Y (A :length 2))
(definterleaved (Z) (X Y))
(defconstraint tmp () (vanishes! (- A Z)))

@DavePearce DavePearce linked a pull request Oct 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants