Things to remove from core

These files and folders might be removed

• scilab/
• PluginsInfo.md
• prova.txt
• demos/demo_wolf_imorted_graph.cpp
• demos/demo_analytic_autodiff_factor.cpp
• yaml files associated to the demos above (3 files in demos/)
• demos/solver/

@joanvallve opinions?

EDIT:

For this MR:

Keep:

• demos/demo_wolf_imorted_graph.cpp
• demos/demo_analytic_autodiff_factor.cpp
• yaml files associated to the demos above (3 files in demos/)
• demos/solver/

Remove:

• scilab/
• PluginsInfo.md
• prova.txt

added Agreement required Code organization Opinions wanted labels

assigned to @jsola

Regarding the demos, we agreen on recycle them since they are useful (the first implements how to import a graph file used by old SLAM benchmarks, and the second compares analytic and autodiff factors for 2D odometry) #409. Of course we can revise the agreement!

About /demos/solver... if there is someone that wants to try implementing a WOLF solver, it can be useful... otherwise, we can remove everything there.

I think the solver deserves a plugin to rest there, but the likelihood that someone uses it in core is so small that having it in core makes no sense.

The plugin, well, just for having it somewhere. Otherwise a tag in this repo and then delete it.

For this MR:

Keep:

• demos/demo_wolf_imorted_graph.cpp
• demos/demo_analytic_autodiff_factor.cpp
• yaml files associated to the demos above (3 files in demos/)
• demos/solver/

Remove:

• scilab/
• PluginsInfo.md
• prova.txt

created merge request !438 (closed) to address this issue

mentioned in merge request !438 (closed)

changed the description