[CI] Edit the notify setting in our CI #76
Merged
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.
This pull request introduces a number of changes primarily aimed at refactoring the
bitblas/ops/impl/batch_matmul_impl.py
file for better code organization and maintainability. The changes also include modifications to the Continuous Integration (CI) workflow and some minor adjustments in other files.Refactoring of
bitblas/ops/impl/batch_matmul_impl.py
:matmul_nt
andmatmul
have been replaced with two new classes:BatchMatMulEmitter
andBatchMatMulSelector
. These classes encapsulate the same functionality but provide a more structured and maintainable approach.BatchMatMulEmitter
class is responsible for emitting the TIR script for batch matrix multiplication. It includes methods for validating dimensions, applying bias, and converting data types.BatchMatMulSelector
class is responsible for selecting the appropriate TIR script emitter based on the provided layout and propagation parameters.select_implementation
function has been updated to use the newBatchMatMulSelector
class.Changes in other files:
notify
job has been added to the CI workflow in.github/workflows/ci.yml
. This job runs when previous jobs fail and prints a message indicating that the jobs have failed but no email will be sent.bitblas/ops/impl/__init__.py
, anoqa: F401
comment has been added to the import statement fortir_interleave_weight
. This prevents linters from flagging the imported but unusedtir_interleave_weight
as an error.bitblas/ops/impl/base.py
has been created to provide two abstract base classesTIRScriptEmitter
andTIRScriptSelector
for TIR script emitters and selectors, respectively.