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

DynaFed note for con_max_block_sig_size #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

opacey
Copy link

@opacey opacey commented Apr 1, 2022

Included a note that the signature size constant under DynaFed is 77, rather than 74 prior to DynaFed. The formula for con_max_block_sig_size is affected. The guide doesnt yet use DynaFed so this may not be the most appropriate place for this note, feel free to reject it and put in a more appropriate location.

Included a note that the signature size constant under DynaFed is 77, rather than 74 prior to DynaFed. The formula for con_max_block_sig_size is affected. The guide doesnt yet use DynaFed so this may not be the most appropriate place for this note, feel free to reject it and put in a more appropriate location.
@wintercooled
Copy link
Collaborator

Thanks @opacey - I will check with the elements team who did Dyna fed and merge when I've had this confirmed. Thanks for the Pull Request and drawing attention to it.

@stevenroose
Copy link
Member

Where did you get the 77 from?

@opacey
Copy link
Author

opacey commented Apr 5, 2022

Only from a process of trial and error. 77 seems to work in place of 74 in the pre-dynafed formula when applied to custom chains. Smaller values cause errors, and larger dont - this was the smallest which worked so I assume it is correct. However perhaps the formula itself is now different and it is only coincidence that 77 lands on a big enough final value. Sorry, should have been more clear in the original comment that this was speculation.

@wintercooled
Copy link
Collaborator

@stevenroose - is this ok to merge after the conversation above or is there a set formula that derives the figure?

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