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.

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

fix convesion to bytes error line 65 in delegate_info.py #2573

Closed
wants to merge 1 commit into from

Conversation

Assh-codes
Copy link
Contributor

Bug fix

properly convert the list, causing this error in delegate_info.py line 65:
argument 'encoded': 'list' object cannot be converted to 'PyBytes'

Welcome!

Due to GitHub limitations,
please switch to Preview for links to render properly.

Please choose the right template for your pull request:

properly convert the list, causing this error:
argument 'encoded': 'list' object cannot be converted to 'PyBytes'
@thewhaleking
Copy link
Contributor

Hey @Assh-codes thanks for the PR. There's two things, however:

  1. All PRs should be against the staging branch.
  2. This method already expects bytes, so the issue actually lies in the Subtensor.get_delegates that sends hex to this method, rather than this method itself. You can see where this is done correctly in AsyncSubtensor.get_delegates.

Let me know if you'd like to update this PR or start a new one.

@thewhaleking
Copy link
Contributor

Sorry, actually looked into it, and this is already fixed in staging: #2551

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