-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feat/subnet sync #37
Feat/subnet sync #37
Conversation
migrations.RemoveField( | ||
model_name="uploadedfile", | ||
name="user", | ||
), |
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.
this was merged with another PR
|
||
class Migration(migrations.Migration): | ||
dependencies = [ | ||
("core", "0005_alter_validatorinstance_server_and_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.
this probably needs adjustment
def send_messages(subnet, subnet_identifier): | ||
""" | ||
This function sends messages to subnet operators. | ||
Args: subnet: Subnet object | ||
subnet_identifier: SubnetID | ||
""" | ||
# send message to subnet operators | ||
pass |
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.
this was implemented in another PR already
from auto_validator.core.utils.utils import get_subnet_by_hotkey, get_user_ip, send_messages | ||
|
||
|
||
def verify_signature_and_route_subnet(view_func): |
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.
I think we might already have it but we may be missing some parameter here
|
||
V1_FILES_URL = "/api/v1/files/" | ||
|
||
|
||
def test_file_upload(api_client, user, eq): | ||
@pytest.mark.django_db | ||
def test_file_upload_with_valid_signature(api_client, wallet, validator_instance): |
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.
tests are cool!
def wallet(): | ||
coldkey_name = "auto-validator7" | ||
hotkey_name = "testhotkey7" | ||
command1 = f"btcli wallet new_coldkey --wallet.name {coldkey_name}" |
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.
it'd be better to call proper functions from bittensor, but if it aint broken, don't fix it
added subnet synchronization with side-by-side comparison