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

[Target] Improve TVM Target related items #45

Merged
merged 2 commits into from
Jun 1, 2024

Conversation

LeiWang1999
Copy link
Contributor

This pull request primarily focuses on improving the target detection functionality in the bitblas Python package, particularly for NVIDIA targets. The changes include the addition of a new function to list all NVIDIA targets, modifications to error handling and logging, and a new script to display available targets. Additionally, a new dependency, tabulate, has been added to the development requirements.

Target detection improvements:

  • python/bitblas/utils/__init__.py: The get_all_nvidia_targets function has been imported from the target_detector module.
  • python/bitblas/utils/target_detector.py: A new function get_all_nvidia_targets has been added to list all available NVIDIA targets. Additionally, the find_best_match function has been updated to provide a more informative error message when the best match is not a valid CUDA target. [1] [2]

New script for displaying targets:

  • tools/get_available_targets.py: A new script has been added that uses the get_all_nvidia_targets function to retrieve all available NVIDIA targets and display them in a table format.

Dependency changes:

  • requirements-dev.txt: The tabulate package has been added to the development requirements to support the new script for displaying targets.

TODO Items:

  • Related documents for the usage of targets.

@LeiWang1999 LeiWang1999 merged commit efab450 into microsoft:main Jun 1, 2024
4 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.

1 participant