Skip to content

Update README with RL-ready datasets information#78

Merged
kenjiro-mk merged 2 commits intomasterfrom
akshatgarg06-1
Mar 4, 2026
Merged

Update README with RL-ready datasets information#78
kenjiro-mk merged 2 commits intomasterfrom
akshatgarg06-1

Conversation

@akshatgarg06
Copy link
Copy Markdown
Collaborator

Added section on RL-ready datasets for DQN and QMIX.

Added section on RL-ready datasets for DQN and QMIX.
@github-actions
Copy link
Copy Markdown

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.10

@github-actions
Copy link
Copy Markdown

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.9

@github-actions
Copy link
Copy Markdown

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.8

Copy link
Copy Markdown
Collaborator

@kenjiro-mk kenjiro-mk left a comment

Choose a reason for hiding this comment

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

Is "RL-ready" the name of this process? What is the origin of "ready"?

Good catch — “RL-ready” is not a formal pipeline name in this repo.
I used it informally to mean “formatted so RL models can consume it directly” (fixed tensor shapes, padding, masks, splits, and action ids).
To make this explicit, I renamed the wording to “SAR-to-RL Dataset Conversion (DQN / QMIX)” in the docs and script description, so it reflects the actual step: converting SAR events.jsonl into model input datasets.
@akshatgarg06
Copy link
Copy Markdown
Collaborator Author

@kenjiro-mk
I used it informally to mean “formatted so RL models can consume it directly” (fixed tensor shapes, padding, masks, splits, and action ids).
To make this explicit, I renamed the wording to “SAR-to-RL Dataset Conversion (DQN / QMIX)” in the docs and script description, so it reflects the actual step: converting SAR events.jsonl into model input datasets.

@akshatgarg06 akshatgarg06 requested a review from kenjiro-mk March 4, 2026 06:12
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.8

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.10

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.9

@kenjiro-mk kenjiro-mk merged commit 8160aa2 into master Mar 4, 2026
3 checks passed
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.

2 participants