Skip to content

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
PINTO0309 authored Jul 2, 2023
1 parent d50774b commit 31f9348
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,12 @@
2. Abolish the narrowing of feature points by score (heavy use of `NonZero` judgment) and change to a fixed extraction of the top 20 scores.
3. Although the feature points are insufficiently narrowed down by score, only 20 of the inference results need to be filtered by score.
4. The program should determine the score thresholds. For example, use Numpy.
5. Inference performance is only slightly worse because 20 fixed points are needlessly processed.
6. Since it is a pain to write preprocessing of the input image in the program, grayscale conversion is included in the model.
5. The process of removing feature points contained in the top, bottom, left, and right 4 pixels near the edges of the image has been removed from the onnx. This is to eliminate from onnx the `NonZero` processing that sacrifices inference performance and generality of the model.
6. Inference performance is only slightly worse because 20 fixed points are needlessly processed.
7. Since it is a pain to write preprocessing of the input image in the program, grayscale conversion is included in the model.
![image](https://github.com/PINTO0309/LightGlue-ONNX/assets/33194443/f6980a01-fea9-42f7-a74d-deff7a902cab)
7. Since my implementation is only temporary and fabio-sim seems to be improving the functionality very frequently, I think it is more reasonable to wait for fabio-sim to improve the functionality.
8. All OPs can be converted to TensorRT Engine. It will be a highly efficient model that is not offloaded to the CPU.
8. Since my implementation is only temporary and fabio-sim seems to be improving the functionality very frequently, I think it is more reasonable to wait for fabio-sim to improve the functionality.
9. All OPs can be converted to TensorRT Engine. It will be a highly efficient model that is not offloaded to the CPU.
![image](https://github.com/PINTO0309/LightGlue-ONNX/assets/33194443/84feb402-a3d5-47f6-a34e-62b5d40b3127)
![image](https://github.com/PINTO0309/LightGlue-ONNX/assets/33194443/bcfec240-2868-42dd-8f3e-1b4169ff3010)

Expand Down

0 comments on commit 31f9348

Please sign in to comment.