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.

Sign up for GitHub

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

Jump to bottom

Tests for metagraph, highlighting bug #2203

Open
wants to merge 9 commits into
base: staging
Choose a base branch
from
Open

Tests for metagraph, highlighting bug #2203

wants to merge 9 commits into from

Commits on Aug 29, 2024

  1. Child Hotkeys netuid Refactor

    @opendansor
    opendansor committed Aug 29, 2024
    Configuration menu
    Copy the full SHA
    0d236db View commit details
    Browse the repository at this point in the history

  2. CHK Test

    @opendansor
    opendansor committed Aug 29, 2024
    Configuration menu
    Copy the full SHA
    4379c59 View commit details
    Browse the repository at this point in the history

  3. u16 float limit

    @opendansor
    opendansor committed Aug 29, 2024
    Configuration menu
    Copy the full SHA
    4b046c3 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'staging' into feat/opendansor/chk

    @opendansor
    opendansor authored Aug 29, 2024
    Configuration menu
    Copy the full SHA
    dfe3cfd View commit details
    Browse the repository at this point in the history

  5. Merge pull request opentensor#2276 from opentensor/feat/opendansor/chk 

    Child Hotkeys netuid Refactor
    @opendansor
    opendansor authored Aug 29, 2024
    Configuration menu
    Copy the full SHA
    f0e09f2 View commit details
    Browse the repository at this point in the history

Commits on Sep 3, 2024

  1. test_metagraph.py: fix assertions 

    The following does not do what the author probably thought:

   assert 'a' and 'b' in s

as it evaluates as:

   assert 'a' and ('b' in s)

and hence:

   assert 'b' in s

This is fixed by splitting the assertions in two or by changing the
condition to ('a' in s and 'b' in s).
    μ committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    c7c6c5d View commit details
    Browse the repository at this point in the history

  2. test_metagraph.py: replace magic constant with netuid 

    Using variable netuid instead of magic constant 1 (which happens to be
the netuid of the added subnet) provides context to the usage of the
value 1 and makes the test more future proof.
    μ committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    60bc47a View commit details
    Browse the repository at this point in the history

  3. test_metagraph.py: test .save() and .load() on metagraph 

    Saving and loading should transfer the complete state of the metagraph.
    μ committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    c560a46 View commit details
    Browse the repository at this point in the history

  4. test_metagraph.py: test adding multiple subnets and using netuid 2 

    A minor tweak to the test that increases coverage of the test.
    μ committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    b63fae3 View commit details
    Browse the repository at this point in the history