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

Support colons in input-shapes tvmc command line arguments #9080

Merged
merged 1 commit into from
Sep 24, 2021

Conversation

binarybana
Copy link
Contributor

@binarybana binarybana commented Sep 23, 2021

Support colons in input names passed into input-shapes tvmc command line arguments. Ideally we'd simplify the format so that it's not a complicated regex, but wanted to address an issue that I ran into in the wild.

@Mousius
Copy link
Member

Mousius commented Sep 23, 2021

Hi @binarybana, the changes you've pushed don't seem related to the changes you describe, could you take another look that this PR? 😸

@binarybana binarybana changed the title Some rust cleanups Support colons in input-shapes tvmc command line arguments Sep 23, 2021
@binarybana
Copy link
Contributor Author

Apologies for that! Got sloppy with hub last night. Updated.

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for the fix!

Copy link
Member

@Mousius Mousius left a comment

Choose a reason for hiding this comment

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

No worries, makes sense to me 😸

@leandron leandron merged commit af3c819 into apache:main Sep 24, 2021
@leandron
Copy link
Contributor

Merged now, thanks @Mousius @binarybana @mbrookhart!

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.

4 participants