IsaacLab icon indicating copy to clipboard operation
IsaacLab copied to clipboard

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

Open WYYAHYT opened this issue 1 year ago • 1 comments

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 avatar Sep 26 '24 07:09 WYYAHYT

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

WYYAHYT avatar Sep 26 '24 07:09 WYYAHYT

Thank you for posting this. We will review.

RandomOakForest avatar Jan 05 '25 09:01 RandomOakForest