Skip to content

change obs and action to dict - #95

Open
jxx123 wants to merge 5 commits into
masterfrom
interface_update
Open

change obs and action to dict#95
jxx123 wants to merge 5 commits into
masterfrom
interface_update

Conversation

@jxx123

@jxx123 jxx123 commented Apr 12, 2026

Copy link
Copy Markdown
Owner

No description provided.

jxx123 and others added 5 commits April 12, 2026 14:40
…ollouts

Add an optional `env_seeds` param threaded through T1DSimVectorEnv ->
T1DPatientBatch, CGMSensorBatch, and BatchScenario. When provided, each
env draws its meals, sensor noise, and initial BG from its OWN RNG seeded
by env_seeds[i], so:

  - envs sharing a seed are bit-identical (meals + noise + init BG),
  - a rollout is a pure function of its per-env seed (reproducible across
    runs and across in-place reset()),
  - envs with different seeds diverge as before.

This makes the vectorized env safe for group-based RL (e.g. GRPO/RLOO):
a group of copies can share one seed so trajectories differ ONLY by the
policy's actions. Default (env_seeds=None) keeps the original fast
single-RNG behavior unchanged.

Verified: same-seed group -> identical BG+CGM; different seeds -> divergent;
reset-in-place reproduces; auto_reset=False freezes terminated envs.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VRhVcpNkjACm3KMGjPjLjx
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