Revolution #1450
Merged
Revolution #1450
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…into revolution
… also the forward
…into revolution
…into revolution
* standardize bittensor imports * run black * standardize typehints * run black and add Calalble * add responses * fix typehints
* remove unnecessary traces * added blacklist reason to trace --------- Co-authored-by: isabella618033 <isabella618033@gmail.com>
* (WIP) Refactor synapse and dendrite to verify body hash in signature * EUREKA! * cleanup print statements * remove (now) unnecessary metaclass override protection * remove metaclass from Synapse def * fix commit * remove print statements * run black * add typehints for request * (WIP) update synapse tests * move hash to utils * remove extra logging * fix repackaging request, inspect body and hash to compare against header hashes * remove streaming example, moved to subnet-1 template * put blacklist first so we don't load the body if we're gonna blacklist anyway * fix axon tests * fix bt. import * fix typehint bt. * fix test relative to header changes * run black * add reason to error logging * create dependency for all forward_fn routes to verify body before passing to synapse function * fix test (again) * add md5 hash option * remove extra trace
add check for missing field
* Add dereg stake to overview command * fix typo * use default dict instead * pass lambda for default factory * remove lambda arg in def factory * run black --------- Co-authored-by: ifrit98 <ifrit98@gmail.com>
Updates command examples to new format
root metagraph fixes
* simplify hashing for request verification * add override protection for property * run black * update synapse tests * update default json b64 value for hash_fields * remove print statement * run black
* merge master into revolution * remove depreicated content
update dockerfile to pytorch dep 2.0.1
* fix for metagraph root weights * black
* add no_prompt confirmation for wallet update command * fix bool flag bug in cli (argparse) * unskip tests * use instantiated subtensor for root and senate incase cli.config is None * run black * unskip and update cli tests * adds simple test for btcli root senate_vote * black --------- Co-authored-by: p-ferreira <pe_drojunior@hotmail.com>
* rm md5 -> sha3 * pass required_hash_fields as attach argument * make default_verify async * move req_hash_fields to a field instead of prop * black * make verify_body_integirty member of self to accesss required_hash_fields * correct default value * proper request name for key * update tests * black * parse required_hash_fields directly from protocol definition itself * run black * move parsing req_hash_fields below signature checks * remove arg docstring for attach()
* fixes for root=True * run black --------- Co-authored-by: ifrit98 <ifrit98@gmail.com>
Eugene-hu
approved these changes
Oct 2, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.