Skip to content

feat(mujoco): add SO-101 mujoco simulation plugin - #292

Open
maxxgx wants to merge 1 commit into
mainfrom
max/mujoco-plugin
Open

maxxgx wants to merge 1 commit into
mainfrom
max/mujoco-plugin

Conversation

@maxxgx

@maxxgx maxxgx commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Copilot AI lite review requested due to automatic review settings September 23, 2026 14:30
@maxxgx
maxxgx requested review from a team as code owners September 23, 2026 14:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Returns:
The process's command line, or ``None`` when it can't be read.
"""
import subprocess # noqa: PLC0415, S404
Comment on lines +474 to +475
result = subprocess.run( # noqa: S603
["ps", "-o", "args=", "-p", str(pid)], # noqa: S607
Comment on lines +474 to +475
result = subprocess.run( # noqa: S603
["ps", "-o", "args=", "-p", str(pid)], # noqa: S607
Returns:
Matching PIDs, with this process and its parent removed.
"""
import subprocess # noqa: PLC0415, S404
Comment on lines +530 to +531
result = subprocess.run( # noqa: S603
["pgrep", "-f", pattern], # noqa: S607
Comment on lines +530 to +531
result = subprocess.run( # noqa: S603
["pgrep", "-f", pattern], # noqa: S607
# host/port are local CLI args, scheme hardcoded to http
request = urllib.request.Request(f"http://{host}:{port}/shutdown", method="POST")
try:
with urllib.request.urlopen(request, timeout=5): # nosec B310 # noqa: S310

request = urllib.request.Request(f"http://{host}:{port}/") # host/port are local CLI args
try:
with urllib.request.urlopen(request, timeout=5) as response: # nosec B310 # noqa: S310

This branch has not been deployed

No deployments
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.

3 participants