Skip to content
This repository was archived by the owner on Jul 10, 2026. It is now read-only.

metagraph sync warning update. avoid potential crash - #1328

Merged
ifrit98 merged 1 commit into
text_promptingfrom
metagraph_sync_warn
May 15, 2023
Merged

metagraph sync warning update. avoid potential crash#1328
ifrit98 merged 1 commit into
text_promptingfrom
metagraph_sync_warn

Conversation

@ifrit98

@ifrit98 ifrit98 commented May 14, 2023

Copy link
Copy Markdown
Contributor

For some reason, if on full sync and there are no neurons. the torch.stack() calls would result in an uncaught exception.

Figured we should log/warn the user this is happening or we can raise an exception if desired. Just ran into this while writing the unittest for the metagraph.

@ifrit98
ifrit98 requested review from Eugene-hu and unconst May 14, 2023 22:16

@Eugene-hu Eugene-hu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ifrit98
ifrit98 merged commit ac1bd25 into text_prompting May 15, 2023
@ifrit98
ifrit98 deleted the metagraph_sync_warn branch May 30, 2023 12:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants