Skip to content

Added Aliked support for onnx export - #103

Open
rohankhaire-work wants to merge 3 commits into
fabio-sim:mainfrom
rohankhaire-work:aliked_support
Open

Added Aliked support for onnx export#103
rohankhaire-work wants to merge 3 commits into
fabio-sim:mainfrom
rohankhaire-work:aliked_support

Conversation

@rohankhaire-work

Copy link
Copy Markdown

This Pull request references the issue #69. I have added onnx export for aliked.

@manhtd20z

Copy link
Copy Markdown

Do you think aliked, superpoint or sift, which is better extractor ?

@ahojnnes

ahojnnes commented Aug 25, 2025

Copy link
Copy Markdown

This doesn't work for dynamic input image sizes for me. I get errors like these when passing images with different input dimension:

E:onnxruntime:, sequential_executor.cc:572 ExecuteKernel] Non-zero status code returned while running Reshape node. Name:'/block3/conv1/Reshape_5' Status Message: /private/tmp/onnxruntime-20250819-46528-71tomy/onnxruntime/core/providers/cpu/tensor/reshape_helper.h:47 onnxruntime::ReshapeHelper::ReshapeHelper(const TensorShape &, TensorShapeVector &, bool) input_shape_size == size was false. The input tensor cannot be reshaped to the requested shape. Input shape:{1,32,66,66}, requested shape:{1,1,32,50,66}

@manhtd20z

Copy link
Copy Markdown

This doesn't work for dynamic input image sizes for me. I get errors like these when passing images with different input dimension:

E:onnxruntime:, sequential_executor.cc:572 ExecuteKernel] Non-zero status code returned while running Reshape node. Name:'/block3/conv1/Reshape_5' Status Message: /private/tmp/onnxruntime-20250819-46528-71tomy/onnxruntime/core/providers/cpu/tensor/reshape_helper.h:47 onnxruntime::ReshapeHelper::ReshapeHelper(const TensorShape &, TensorShapeVector &, bool) input_shape_size == size was false. The input tensor cannot be reshaped to the requested shape. Input shape:{1,32,66,66}, requested shape:{1,1,32,50,66}

it's because the shape converted is fixed with number of keypoint output. seem like we need add padding static

@ahojnnes

Copy link
Copy Markdown

@rohankhaire-work @manhtd20z Any idea how to solve the issue?

@manhtd20z

Copy link
Copy Markdown

just using threshold is -1. The error will solve.

@ahojnnes

Copy link
Copy Markdown

@manhtd20z Your suggested fix does not work for me, assuming you mean the max_num_keypoints as the "threshold"?

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.

3 participants