-
Notifications
You must be signed in to change notification settings - Fork 9
uid trimming #95
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
uid trimming #95
Conversation
Deploying developer-docs with
|
Latest commit: |
11764b7
|
Status: | ✅ Deploy successful! |
Preview URL: | https://d30deb83.developer-docs-6uq.pages.dev |
Branch Preview URL: | https://uid-trimming.developer-docs-6uq.pages.dev |
docs/subnets/uid-trimming.md
Outdated
|
||
:::info | ||
|
||
- The minimum UID count to which subnet owners can trim is currently 256. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be the minimum or maximum?
AFAIK, the idea was to be able to have lower than the default 256 or am i missing something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like they changed it to 64!
https://discord.com/channels/1120750674595024897/1358440760013361192/1417872187909472348
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmmm
problem is i still haven't been able to test this locally because of the fastblocks issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the changes are still on btcli staging; not sure there's a way to test.
had to go through the trimming PR on the btcli repo to confirm.
No description provided.