facebookresearch / habitat-sim

A flexible, high-performance 3D simulator for Embodied AI research.
https://aihabitat.org/
MIT License
2.45k stars 404 forks source link

--Configuration : Add ability to handle vectors of fields within subconfigs #2394

Closed jturner65 closed 1 month ago

jturner65 commented 1 month ago

Motivation and Context

This PR takes the support for vectors of points introduced with MarkerSets, promotes it to the underlying Configuration and expands it to be vectors of any type. In a nutshell a vector of data can be stored in a subconfig by having the key be a string rep of the index in the vector referencing the value.

This obviates the need for local vectors to implementers of the Configuration class, such as SemanticAttributes, which stored the polyloop of points in a vector member varaiable.

How Has This Been Tested

Locally c++ and python tests pass.

Types of changes

Checklist