[Dev] Refactor scripts based on our new directory structure #69
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ref to the pull request #68 , our file structure has been changed, the scripts and tvm related import should be modified as well.
TODO Items:
This pull request includes changes to the
bitblas
module, primarily focused on modifying the import paths for thetvm
module and refactoring thebitblas/__init__.py
andbitblas/ops/general_matmul.py
files. The changes are aimed at improving the structure and clarity of the code.Changes to import paths:
tvm
module have been changed fromimport tvm
tofrom bitblas import tvm
in multiple files. This change suggests a restructuring of the module's dependencies and is aimed at improving the clarity and maintainability of the code. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18]Changes to
bitblas/__init__.py
:tvm
module in theinstall_tvm_path
anddevelop_tvm_path
variables has been updated. The changes suggest a restructuring of the module's directory.try_inline_contiguous_spatial
function. This change is likely meant to improve the readability of the code.Changes to
bitblas/ops/general_matmul.py
:warmup
method in thebitblas/module/__init__.py
file has been refactored. The changes include reordering of the code and changes to the arguments passed to theself.bitblas_matmul.lib.call
method. These changes are likely aimed at improving the efficiency and readability of the code.__legalize_propagate
method has been updated to handle a new case wherepropagate_b
isNonTransform
. This change is likely meant to improve the functionality of the method.