Skip to content

Update aztec_code_generator.py#5

Merged
delimitry merged 2 commits intodelimitry:masterfrom
jravallec:patch-1
Jul 28, 2020
Merged

Update aztec_code_generator.py#5
delimitry merged 2 commits intodelimitry:masterfrom
jravallec:patch-1

Conversation

@jravallec
Copy link
Copy Markdown
Contributor

Reviewed part when leaving binary mode to any other mode.
Issue for the number of bytes when superior to 31 corrected.

Reviewed part when leaving binary mode to any other mode.
Issue for the number of bytes when superior to 31 corrected.
@delimitry delimitry merged commit d893f3e into delimitry:master Jul 28, 2020
dlenski added a commit to dlenski/aztec_code_generator that referenced this pull request Oct 19, 2020
Assuming that new sequences after delimitry#5 are indeed optimal.
dlenski added a commit to dlenski/aztec_code_generator that referenced this pull request Oct 19, 2020
@dlenski
Copy link
Copy Markdown

dlenski commented Oct 19, 2020

The tests were broken by this change. I fixed them in my fork (which also adds Python packaging), in dlenski@f2bcd2d … based on the assumption that the new sequences resulting from this PR are indeed the correct ones.

@delimitry
Copy link
Copy Markdown
Owner

@dlenski Thank you!

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.

3 participants