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

--Move AssetType enum class attributes namespace #2386

Closed jturner65 closed 2 months ago

jturner65 commented 2 months ago

Motivation and Context

This PR moves the AssetType enum from the asset namespace to the metadata::attributes namespace where it can reside with the other metadata-related enum classes in AttributesEnumMaps. This is primarily for consistency but also to facilitate interactions with the enum-backed data in the Attributes that consume it.

The PR also moves the EMPTY_SCENE definition to Esp.h from Asset.h, in an effort to also maintain consistency in where our system-wide consts are stored.

How Has This Been Tested

Locally c++ and python tests pass

Types of changes

Checklist