Skip to content

Metagraph Improvements#2659

Merged
basfroman merged 2 commits intofeat/roman/add-a-few-fields-to-metagrapg-instancefrom
fix/thewhaleking/fix-async-torch-metagraph
Feb 11, 2025
Merged

Metagraph Improvements#2659
basfroman merged 2 commits intofeat/roman/add-a-few-fields-to-metagrapg-instancefrom
fix/thewhaleking/fix-async-torch-metagraph

Conversation

@thewhaleking
Copy link
Contributor

  • Fixes Torch AsyncMetagraph properties to match those of sync torch version
  • Moves shared __init__ items to the __init__ of the Mixin
  • self.lite is now set at __init__
  • Metagraph.sync and AsyncMetagraph.sync now correctly use the lite designation at instantiation by default (i.e. if you create the metagraph with metagraph = Metagraph(1, lite=False), calling metagraph.sync() will now use lite = False

…nc` to coincide with the creation of the subtensor object if not specified.
@thewhaleking thewhaleking requested a review from a team February 11, 2025 17:04
@basfroman basfroman merged commit 4f2b7f7 into feat/roman/add-a-few-fields-to-metagrapg-instance Feb 11, 2025
15 checks passed
@basfroman basfroman deleted the fix/thewhaleking/fix-async-torch-metagraph branch February 11, 2025 17:48
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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants