Skip to content

Merge async-pre-merge-new-async to staging#2656

Merged
basfroman merged 572 commits intostagingfrom
merge/roman/async-pre-merge-new-async-to-staging
Feb 11, 2025
Merged

Merge async-pre-merge-new-async to staging#2656
basfroman merged 572 commits intostagingfrom
merge/roman/async-pre-merge-new-async-to-staging

Conversation

@basfroman
Copy link
Collaborator

@basfroman basfroman commented Feb 10, 2025

Base for e2e tests is subtensor/tree/devnet-ready

Roman and others added 30 commits January 28, 2025 15:03
…nsor.commit

Add alias for `Subtensor.commit` as `set_commitment`
…raph

[RAO] Add methods to fetch metagraph data from the chain
…class

Update metagraph class with `rao` stuff
@basfroman basfroman changed the title merge/roman/async-pre-merge-new-async-to-staging Merge async-pre-merge-new-async to staging Feb 11, 2025
@basfroman basfroman requested a review from a team February 11, 2025 03:01
Roman and others added 21 commits February 11, 2025 08:27
…nc` to coincide with the creation of the subtensor object if not specified.
…to-metagrapg-instance

add name and symbol fields to metagraph
…ne_chain_endpoint_and_network

Using one determine_chain_endpoint_and_network
…es_teardown

Tests: separate `templates` fixture
…-merge-new-async-to-staging

# Conflicts:
#	tests/e2e_tests/test_incentive.py
Copy link
Contributor

@thewhaleking thewhaleking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM


return info_dictionary
class AsyncSubtensor(SubtensorMixin):
"""Thin layer for interacting with Substrate Interface. Mostly a collection of frequently-used calls."""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docstring.

Comment on lines +112 to +113
Raises:
Any exceptions raised during the setup, configuration, or connection process.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove.

@basfroman basfroman merged commit 820b1e2 into staging Feb 11, 2025
31 checks passed
@basfroman basfroman deleted the merge/roman/async-pre-merge-new-async-to-staging branch February 11, 2025 22:47
This was referenced Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge rao RAO related changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants