Skip to content

Conversation

@kparzysz-quic
Copy link
Contributor

Type annotations don't do anything, the type conversion needs to be explicit.

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Type annotations don't do anything, the type conversion needs to be
explicit.
@kparzysz-quic
Copy link
Contributor Author

cc: @mehrdadh

@kparzysz-quic
Copy link
Contributor Author

The reason for this is that if someone wants to initialize HEXAGON_RPC_DIR, like the other globals, they'd have to to that explicitly with a pathlib.Path, but the most natural way would be to do that with a string. If they do that with a string, they'll end up returning str from this function.

Copy link
Member

@mehrdadh mehrdadh left a comment

Choose a reason for hiding this comment

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

Thanks for this change! LGTM

@masahi masahi merged commit da3b63e into apache:main Jan 19, 2022
@kparzysz-quic kparzysz-quic deleted the hexagon-pathlib branch January 19, 2022 14:34
yuanfz98 pushed a commit to yuanfz98/tvm that referenced this pull request Jan 24, 2022
Type annotations don't do anything, the type conversion needs to be
explicit.
crazydemo pushed a commit to crazydemo/tvm that referenced this pull request Jan 27, 2022
Type annotations don't do anything, the type conversion needs to be
explicit.
ylc pushed a commit to ylc/tvm that referenced this pull request Feb 16, 2022
Type annotations don't do anything, the type conversion needs to be
explicit.
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