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

Amrita/tvm #1

Merged
merged 8 commits into from
Mar 17, 2021
Merged

Amrita/tvm #1

merged 8 commits into from
Mar 17, 2021

Conversation

amritaravishankar
Copy link
Owner

@amritaravishankar amritaravishankar commented Mar 14, 2021

Closes cap-ntu#137 - Adding a new conversion tool - TVM

What has been done to verify that this works as intended?

A unit testing file has been created to verify the conversion function works as intended. A Dockerfile for the TVM environment has been created as well.

Why is this the best possible solution? Were any other approaches considered?

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

The change allows users to perform an additional conversion to TVM

Does this change require updates to documentation?

Yes, documentation regarding TVM installation and setting up is required

Before submitting this PR, please make sure you have:

  • run python -m pytest tests/ and confirmed all checks still pass.
  • verified that any code or assets from external sources are properly credited.

@amritaravishankar amritaravishankar merged commit 4a43128 into master Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant