[Proposal] "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math.py" is overriding the stdlib module "math"

isaac-sim / IsaacLab

Unified framework for robot learning built on NVIDIA Isaac Sim

https://isaac-sim.github.io/IsaacLab

Other

2.15k stars 880 forks source link

[Proposal] "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math.py" is overriding the stdlib module "math" #1041

Open WYYAHYT opened 1 month ago

WYYAHYT commented 1 month ago

Proposal

There is a confilct between isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math.py and stdlib math

Motivation

While using VScode, Pylance extension catches this conflict and produces a warning: "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math.py" is overriding the stdlib module "math"

Alternatives

Rename "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math.py" to "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/math_{xx}.py" or any ohter names

WYYAHYT commented 1 month ago

same proposal for "isaacsim/exts/omni.isaac.core/omni/isaac/core/utils/random.py"