Skip to content

feat(memory): select versioned LIBERO corpora by model - #202

Open
ye1457 wants to merge 7 commits into
RLinf:mainfrom
ye1457:feat/libero-memory-versions
Open

ye1457 wants to merge 7 commits into
RLinf:mainfrom
ye1457:feat/libero-memory-versions

Conversation

@ye1457

@ye1457 ye1457 commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

LIBERO selects the GPT-5.5 or Astra memory corpus independently of the running model's effort. CLI, Dashboard, prompts, tools and Flash use the same selected root.

Changes

Validation

@ye1457

ye1457 commented Sep 22, 2026

Copy link
Copy Markdown
Contributor Author

Synced latest main (eb269c8) in aff285e. No manual conflict resolution was needed; a local integration merge with #190 also completed cleanly. Validation: 620 passed / 3 skipped on this PR; 673 passed / 3 skipped with #190. Pre-commit and strict EN/ZH documentation builds passed on both. No force-push or PR merge.

@ye1457

ye1457 commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

Synced main #164 and checked the new evaluate/explore routes in 10c6523. The selected root already reaches prompts, toolkit and Flash correctly; the added checks cover that path and Astra’s unavailable replay assets.

623 tests pass here; the #190/#201/#202 integration has 693 passes, with 3 optional-dependency skips in each. Both actual model/MCP checks read the selected LIBERO corpus successfully. Pre-commit and both docs builds also pass.

@qurakchin qurakchin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Sonorous281 please review this pr.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

conflict in memory.rst:
existing: suite/+task_only/,but the newly added is task-family/+task-specific/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated both guides to use task-family/ and task-specific/ consistently, retaining the full layer descriptions from #190. The frozen reproduction branch is documented separately because it keeps its historical layout. See 003552b and the follow-up documentation commits.

Comment thread robots/libero/memory.py
# See the License for the specific language governing permissions and
# limitations under the License.

"""Model selection and complete, revision-isolated LIBERO memory downloads."""

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why to write LIBERO-specific code here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved the LIBERO selection, cache and replay code under robots/libero/. RobotSpec now provides optional validation and memory-preparation hooks; the shared loader keeps the default path for other robots. CLI and Dashboard tests cover validation before service startup and the selected root. See 003552b and the follow-up documentation commits.

Comment thread rpent/cli/memory.py Outdated
sync = subparsers.add_parser(
"sync", help="Download one model-specific LIBERO memory corpus."
)
sync.add_argument("--robot", choices=["libero"], default="libero")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this file only for libero now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The merge, validate and build-index commands remain shared. The LIBERO download command is now python -m robots.libero.memory sync. Updated both guides and the companion HF README examples. See 003552b and the follow-up documentation commits.

Comment thread robots/libero/memory.py
@ye1457
ye1457 force-pushed the feat/libero-memory-versions branch from 10c6523 to 4a0c218 Compare September 25, 2026 10:09
@ye1457

ye1457 commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor Author

Updated the English and Chinese Flash and memory guides for the complete GPT-5.5 corpus: 78 plan/anchor pairs under GPT_5.5_xhigh/flash/. The examples use that directory and identify the two unavailable tasks. Historical evaluation figures and runtime behavior are unchanged.

The source archive tags were not created. Historical download examples now use the original commit IDs, and the source revisions and hashes remain in the data manifest.

HF #12 and #13 have now been rebased and pushed (b14360f, bddc229). The final candidate passed the 120 related regressions, all 78 Flash loader checks, and RoboCasa memory selection and file-read checks for all 50 tasks at seeds 1 and 5. Pre-commit and fresh strict English/Chinese documentation builds also passed.

Code documentation commits: 112078b, e2ac7d3, af7908d. Merge the data PRs in order (#12, then #13) and coordinate with #190/#202. The four planned model-to-MCP checks against merged HF main remain pending; no simulator evaluation was rerun.

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