Precompute field's index in VectorClassifier (!645) · Merge requests · Main Repositories / otb · GitLab

Snippets Groups Projects

Merged Guillaume Pasero requested to merge 1989-refac-readinputlistsample into develop 5 years ago

All threads resolved!

First try to make VectorClassifier faster at loading geometries.

Related to #1989 (closed)

Activity

Guillaume Pasero changed milestone to %7.1.0 5 years ago

changed milestone to %7.1.0
Guillaume Pasero added CNES backlog To Do + 1 deleted label 5 years ago

added CNES backlog To Do + 1 deleted label
Guillaume Pasero @gpasero started a thread on an old version of the diff 5 years ago

Automatically resolved 5 years ago by Guillaume Pasero
Last reply by Guillaume Pasero 5 years ago

Guillaume Pasero mentioned in merge request !627 (closed) 5 years ago

mentioned in merge request !627 (closed)

Yannick TANGUY @ytanguy 5 years ago

Developer

Can you check if your solution improves performance ? Do we have any test to measure out reading time of such vector data ?

Guillaume Pasero resolved all threads 5 years ago

resolved all threads

Guillaume Pasero added 2 commits 5 years ago

added 2 commits

d1a9186e - BUG: #1989 (closed): expose Feature::GetFieldIndex()
603e3fc1 - BUG: #1989 (closed): keep initial interface for ReadInputListSample

Compare with previous version

Guillaume Pasero @gpasero 5 years ago

Author Reporter

On official test apTvClSVMVectorClassifier (using 2840 samples, with 11 fields):

Before: 2.35 sec
After: 2.08 sec

The gain should be significant on @ArthurV dataset.

Guillaume Pasero merged 5 years ago

merged

Guillaume Pasero mentioned in commit 5 years ago

mentioned in commit 32b26f4c

Please register or sign in to reply