Add new remove_stake_full_limit extrinsic.

[shamil-gadelshin]

Description

This PR adds a new remove_stake_full_limit extrinsic. The extrinsic uses the logic of remove_stake_limit but without partial and 'unstake_amount' parameters. It removes all stakes from the subnet.

Related Issue(s)

• Closes Add additional argument to unstake_all and unstake_all (or remove_all) #1661

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Other (please describe):

Checklist

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[basfroman]

I prepared this PR for Bittensor SDK opentensor/bittensor#2897

As I mentioned in the description, my PR is based on both branches:

• unstake_all changes: https://github.com/opentensor/subtensor/compare/feat/modify-unstake-all
• remove_stake_full_limit changes: https://github.com/opentensor/subtensor/tree/remove_stake_full_limit

To make SDK PR workable, it is necessary to merge the 2 mentioned branches and the devnet-ready with the following changes:

• the unstake_all function must accept 1 additional netuid parameter (as in the first branch)
• the remove_stake_full_limit function works as expected (it needs to be tested deeply). But it accepts a u64 argument limit_price. The SDK itself recalculates rate_tolerance (float) of allowed changes into the limit_price number. This is not a big deal, but we have to do additional call to the chain before the calculation.

Otherwise looks good. Pls keep me updated with changes.

[shamil-gadelshin]

I updated the extrinsic after the offline conversation with @basfroman : I changed the price_limit parameter to optional.

[basfroman]

I updated the extrinsic after the offline conversation with @basfroman : I changed the price_limit parameter to optional.

PR opentensor/bittensor#2897 is ready. I'll wait until your PR is merged to devnet-ready.
SDK implementation works as expected. Thank you!
Testes locally with fast and non-fast-blocks.

[basfroman]

There are no objections from the SDK. Everything is working as discussed.