Update README with new file direction removing the old one#83
Open
akshatgarg06 wants to merge 1 commit intomasterfrom
Open
Update README with new file direction removing the old one#83akshatgarg06 wants to merge 1 commit intomasterfrom
akshatgarg06 wants to merge 1 commit intomasterfrom
Conversation
Collaborator
akshatgarg06
commented
Mar 5, 2026
akshatgarg06
commented
- Removed the conversion script is soccer_sar_to_rl_dataset.py direction.
- SAR2RL expects SAR outputs (events.jsonl) already exist.
Init Test Results 📝
|
Init Test Results 📝
|
Init Test Results 📝
|
Collaborator
Author
Collaborator
Author
|
@kenjiro-mk would this file require any changes, as it depends upon PR #82? |
kenjiro-mk
requested changes
Apr 8, 2026
|
|
||
| - DataStadium | ||
| - Statsbomb with Skillcorner Tracking Data | ||
|
|
Comment on lines
29
to
+41
|
|
||
| ## SAR-to-RL Dataset Conversion (DQN / QMIX) | ||
| This section describes a SAR-to-RL dataset conversion step that formats SAR outputs (`events.jsonl`) into tensors used by | ||
| DQN and QMIX training. This is a preprocessing/data-format step, not a training algorithm. | ||
| The conversion script is `soccer_sar_to_rl_dataset.py`. | ||
|
|
||
| This produces a single shared multi-agent dataset with: | ||
| - `observation`: `(B, T, N, O)` (N=10 attackers) | ||
| - `action`: `(B, T, N)` (discrete action ids; default vocab size 16 with `PAD=15`) | ||
| - `reward`, `done`, `mask`: `(B, T)` | ||
| - `onball_mask`: `(B, T, N)` (for masking unavailable actions) | ||
|
|
||
| ### Run via `SAR_data(...).preprocess_data()` | ||
| You can run SAR2RL through the same entry point as SAR |
Collaborator
There was a problem hiding this comment.
Please add the SAR-to-RL Dataset Conversion (DQN / QMIX) to open-starlab/Document. After that, please add information about RoboCup2D to the Examples section, following the format of the others. Thank you.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.