Skip to content

fix type conversion in axon preprocess #2930

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

Merged
merged 4 commits into from
Jun 23, 2025

Conversation

Windfarer
Copy link

Bug

There's a warning here when the axon get a request.

/home/ubuntu/github/mysubnet/venv/lib/python3.12/site-packages/pydantic/main.py:463: UserWarning: Pydantic serializer warnings:
  PydanticSerializationUnexpectedValue(Expected `int` - serialized value may not be as expected [input_value='9006001', input_type=str])
  return self.__pydantic_serializer__.to_python(
/home/ubuntu/github/mysubnet/venv/lib/python3.12/site-packages/pydantic/main.py:463: UserWarning: Pydantic serializer warnings:
  PydanticSerializationUnexpectedValue(Expected `int` - serialized value may not be as expected [input_value='42476', input_type=str])
  return self.__pydantic_serializer__.to_python(
/home/ubuntu/github/mysubnet/venv/lib/python3.12/site-packages/pydantic/main.py:463: UserWarning: Pydantic serializer warnings:
  PydanticSerializationUnexpectedValue(Expected `int` - serialized value may not be as expected [input_value='42476', input_type=str])
  PydanticSerializationUnexpectedValue(Expected `int` - serialized value may not be as expected [input_value='9006001', input_type=str])
  return self.__pydantic_serializer__.to_python(

9006001 is the version
42476 is the port
these fields are all defined in pydantic as int, but in current code, it has been convert to string.

Description of the Change

convert them to the correct type int

Alternate Designs

Possible Drawbacks

Verification Process

Release Notes

Branch Acknowledgement

[x] I am acknowledging that I am opening this branch against staging

@basfroman basfroman requested a review from a team June 22, 2025 23:27
@basfroman basfroman merged commit 1b8d5c0 into opentensor:staging Jun 23, 2025
125 checks passed
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.

3 participants