Skip to content

Change FGraph structure #58

Open
g-ferrer-sk opened this issue Jan 8, 2021 · 1 comment
Open

Change FGraph structure #58

g-ferrer-sk opened this issue Jan 8, 2021 · 1 comment

Comments

@g-ferrer-sk
Copy link
Contributor

After #29 , there should be a new code structure. Proposals:

  1. Fgraph keeps the abstract classes plus common nodes and all factors are moved to their corresponding places, e.g., PCregistration, VO, IMU, etc.
  2. all included inside FGraph

Maybe it is not necessary to include a /factors' folder, this adds some clutter.

@g-ferrer-sk
Copy link
Contributor Author

Maybe we can remove namespaces if now FGraph will include factors from many different topics, there is no way to differentiate between them, so only mrob

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant