Skip to content

fix: remove backend hot-path type shims - #297

Closed
TATP-233 wants to merge 1 commit into
fix/motrix-remove-unnecessary-np-asarrayfrom
fix/backend-remove-hotpath-type-shims
Closed

TATP-233 wants to merge 1 commit into
fix/motrix-remove-unnecessary-np-asarrayfrom
fix/backend-remove-hotpath-type-shims

Conversation

@TATP-233

Copy link
Copy Markdown
Collaborator

Summary

  • remove backend hot-path np.asarray(...) wrappers that were only serving static typing
  • keep backend return contracts while avoiding extra runtime calls in MuJoCo and Motrix getters
  • use local # type: ignore[no-any-return] only at third-party stub boundaries

Validation

  • pre-commit hooks on commit (ruff, ruff format, pyright, mypy)
  • UV_CACHE_DIR=.uv-cache uv run --no-sync ruff format --check .
  • UV_CACHE_DIR=.uv-cache uv run --no-sync ruff check .
  • UV_CACHE_DIR=.uv-cache uv run --no-sync mypy src/unilab

Notes

@TATP-233

Copy link
Copy Markdown
Collaborator Author

Already included via #293 and merged to main.

@TATP-233 TATP-233 closed this Apr 17, 2026
@TATP-233
TATP-233 deleted the fix/backend-remove-hotpath-type-shims branch April 17, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant