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

[Tests] AsyncSubtensor (Part 5) #2411

Merged
merged 8 commits into from
Nov 13, 2024

Conversation

roman-opentensor
Copy link
Contributor

No description provided.

thewhaleking
thewhaleking previously approved these changes Nov 13, 2024
Base automatically changed from tests/roman/async-unti-tests-part-4 to staging November 13, 2024 17:07
@roman-opentensor roman-opentensor dismissed thewhaleking’s stale review November 13, 2024 17:07

The base branch was changed.

thewhaleking and others added 3 commits November 13, 2024 09:14
* Handles new PasswordError from btwallet.

* Ruff, docstrings.

* Fixes hotkey unlock rather than coldkey

* Added unit test.

* Added unit test. Add integration test.

* Removed coldkeypub

* Opinions.

* More tests

* grammar

* Update test_utils.py

* bump up btwallet version

* fix wording

* fix wording

* Update tests/unit_tests/utils/test_utils.py

Co-authored-by: Cameron Fairchild <[email protected]>

---------

Co-authored-by: Roman <[email protected]>
Co-authored-by: Roman <[email protected]>
Co-authored-by: Cameron Fairchild <[email protected]>
* added tests from `AsyncSubtensor.get_netuids_for_hotkey` until `AsyncSubtensor.neurons_lite`

* ruff

* move out `_decode_hex_identity_dict` from inner function

* added tests until `AsyncSubtensor.query_identity`

* added tests from `AsyncSubtensor.get_netuids_for_hotkey` until `AsyncSubtensor.neurons_lite`

* fix

* ruff
…unti-tests-part-5

# Conflicts:
#	tests/unit_tests/test_async_subtensor.py
@roman-opentensor roman-opentensor merged commit b3f7e85 into staging Nov 13, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants