Skip to content

Fix self.close in case _mock argument used. #109

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
Closed
basfroman opened this issue May 5, 2025 · 1 comment · Fixed by #110
Closed

Fix self.close in case _mock argument used. #109

basfroman opened this issue May 5, 2025 · 1 comment · Fixed by #110
Assignees
@thewhaleking
Labels
bug Something isn't working

Comments

@basfroman
Copy link
Contributor

If we use _mock=True for create Subtensor instance (SubstrateInterface instance inside) then we get the error if we recreate the instance.
del in SubstrateInterface raise the error.
@basfroman basfroman added the bug Something isn't working label May 5, 2025
@thewhaleking
Copy link
Collaborator

Ty

@thewhaleking thewhaleking transferred this issue from opentensor/btcli May 6, 2025
thewhaleking added a commit that referenced this issue May 6, 2025
@thewhaleking
Fixes #109
d0cf9a3
@thewhaleking thewhaleking mentioned this issue May 6, 2025
Merged
@thewhaleking thewhaleking linked a pull request May 6, 2025 that will close this issue
safe __del__ #110
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thewhaleking thewhaleking

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

safe __del__ opentensor/async-substrate-interface
2 participants
@thewhaleking @basfroman