LightConvPoint icon indicating copy to clipboard operation
LightConvPoint copied to clipboard

Is KPconv kinda implemented in this repo?

Open shangguan9191 opened this issue 4 years ago • 3 comments

Dear author,

I notice that part of accuracy result of Lightconv is aggragated from result of KPconv in training. So, if this part of result can be regard as the result of KP implementation?

shangguan9191 avatar Aug 30 '20 16:08 shangguan9191

Hello, No in this repo, we only reimplemented the network architecture of KPConv for K-nearest neighbors conv. layers, not the convolutional layer described in KPConv.

aboulch avatar Sep 02 '20 13:09 aboulch

I am writing a paper comparing different algorithm. The Lightconv and Conv is the fastest algo among all models. I intent to compare KPconv as well. However, i am not a coder. Do you know how use implement also Kpconv for the original layer, in your code style. is it possible that you show some guidance and add the layer in your repo if there is not?

# Network
network:
    model: KPConvCls
    backend_conv: 
        layer: FKAConv(KPConv)
        kernel_separation: false
    backend_search: SearchQuantized

shangguan9191 avatar Sep 03 '20 06:09 shangguan9191

Hello, You cannot do for now. I have coded only the spatial search using K-nearest neighbors (all search algorithm are based on KNN not radius). I do not know if I will have time to do so.

aboulch avatar Sep 16 '20 09:09 aboulch