Skip to content

Commit

Permalink
Fixed typing errors in conv op builder
Browse files Browse the repository at this point in the history
Refine IsOpSupported func and "dilations" param getting

Type: Bug Fix
Signed-off-by: Feiyue Chen <[email protected]>
  • Loading branch information
chenfeiyue-cfy committed Jan 16, 2024
1 parent 46c90e3 commit 6fa7bda
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ namespace vsi {
namespace npu {
class ConvOpBuilder : public BaseOpBuilder {
bool IsOpSupported(const onnxruntime::GraphViewer& graph_viewer,
const Node* node) {
const Node* node) const override {
auto input_defs = node->InputDefs();
auto shape = vsi::npu::util::GetTensorShape(*input_defs[0]);
if (shape.NumDimensions() == 5) {
Expand Down Expand Up @@ -62,7 +62,7 @@ class ConvOpBuilder : public BaseOpBuilder {
helper.Get("strides", is_1d_conv ? std::vector<uint32_t>{default_uint}
: default_vec);
auto dilation =
helper.Get("dilation", is_1d_conv ? std::vector<uint32_t>{default_uint}
helper.Get("dilations", is_1d_conv ? std::vector<uint32_t>{default_uint}
: default_vec);

std::shared_ptr<tim::vx::Operation> op;
Expand Down

0 comments on commit 6fa7bda

Please sign in to comment.