Skip to content

Adds e2e and fixes metagraph save()/load()#2231

Merged
ibraheem-abe merged 3 commits intobtsdkfrom
tests/adds-metagraph-and-extrinsics-e2e
Aug 14, 2024
Merged

Adds e2e and fixes metagraph save()/load()#2231
ibraheem-abe merged 3 commits intobtsdkfrom
tests/adds-metagraph-and-extrinsics-e2e

Conversation

@ibraheem-abe
Copy link
Contributor

@ibraheem-abe ibraheem-abe commented Aug 14, 2024 •
edited
Loading

  • Fixes metagraph save/load where neuron info wasn't getting updated. We've added the field in state_dict and respectively in load_from_path
  • Tweaks existing tests a little bit
  • Adds metagraph e2e test w.r.t new btsdk
  • Adds e2e tests for subtensor functions which will live due to backwards compatibility

@ibraheem-abe ibraheem-abe changed the title Adds e2e for metagraph & subtensor Adds e2e for metagraph & subtensor and fixes metagraph save()/load() Aug 14, 2024
@ibraheem-abe ibraheem-abe changed the title Adds e2e for metagraph & subtensor and fixes metagraph save()/load() Adds e2e and fixes metagraph save()/load() Aug 14, 2024
@ibraheem-abe ibraheem-abe requested a review from a team August 14, 2024 00:36
@ibraheem-abe ibraheem-abe merged commit 115fb7d into btsdk Aug 14, 2024
@ibraheem-abe ibraheem-abe mentioned this pull request Aug 14, 2024
Merged
@ibraheem-abe ibraheem-abe mentioned this pull request Sep 25, 2024
Merged
@thewhaleking thewhaleking deleted the tests/adds-metagraph-and-extrinsics-e2e branch October 2, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@basfroman basfroman basfroman approved these changes

+1 more reviewer

@thealligatorking thealligatorking thealligatorking approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ibraheem-abe @thealligatorking @basfroman