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

[EASY] IVC: attempt to define the real number of columns #2364

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

dannywillems
Copy link
Member

@dannywillems dannywillems commented Jun 18, 2024

And clean old not functional/drafty code

In #2368 and in tests, we use this value to build real world examples.

@dannywillems dannywillems force-pushed the dw/fixing-old-wrong-code-n-col branch 2 times, most recently from d083fdd to feefa5b Compare June 18, 2024 21:19
And clean old not functional/drafty code
@dannywillems dannywillems force-pushed the dw/fixing-old-wrong-code-n-col branch from feefa5b to 6382aef Compare June 18, 2024 21:20
@dannywillems dannywillems changed the title IVC: attempt to write to define the real number of columns [EASY] IVC: attempt to write to define the real number of columns Jun 19, 2024
@dannywillems dannywillems changed the title [EASY] IVC: attempt to write to define the real number of columns [EASY] IVC: attempt to define the real number of columns Jun 20, 2024
@dannywillems dannywillems force-pushed the dw/fixing-old-wrong-code-n-col branch from 16a53e7 to 8800bbb Compare June 24, 2024 09:34
The variable N_COL only defines the number of columns used by the current layout, without quadri.
@dannywillems dannywillems force-pushed the dw/fixing-old-wrong-code-n-col branch from 8800bbb to f0d4e07 Compare June 24, 2024 10:07
@dannywillems dannywillems merged commit 66dc41f into master Jun 24, 2024
6 checks passed
@dannywillems dannywillems deleted the dw/fixing-old-wrong-code-n-col branch June 24, 2024 12:01
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.

None yet

2 participants