Skip to content

docs(repo): remove orphaned pre-re-scope images/ folder (closes #113)#115

Merged
cagataycali merged 1 commit into
strands-labs:mainfrom
yinsong1986:docs/remove-orphaned-images
Jun 19, 2026
Merged

docs(repo): remove orphaned pre-re-scope images/ folder (closes #113)#115
cagataycali merged 1 commit into
strands-labs:mainfrom
yinsong1986:docs/remove-orphaned-images

Conversation

@yinsong1986

Copy link
Copy Markdown
Contributor

What changed

Removes the orphaned root images/ folder (~380 KB) and updates NOTICE
to match.

Deleted:

  • images/architecture_overview.png
  • images/how_it_works_flow.png
  • images/libero_example.mp4

Edited:

  • NOTICE — the "Images and Media" stanza explicitly listed the three
    deleted files. Replaced it with the current originals (docs/assets/strands-logo.svg
    plus the inline Mermaid diagrams) so attribution stays accurate.

Why

These assets were outdated and orphaned:

Test surface

  • hatch run lint — passes (black, isort, flake8).
  • hatch run test203 passed, 33 skipped. No new failures.

This is an asset/attribution-only change; no Python source paths touched.

Acceptance criteria

  • No orphaned, pre-re-scope image assets remain at the repo root.
  • Any retained visual reflects the current Isaac-Sim-plugin
    architecture and is referenced by the README/docs — the Mermaid
    diagrams in docs/index.md + docs/architecture.md already cover the
    architecture/flow visuals; no new static image was introduced.

Out-of-scope follow-ups

  • None. The issue offered an alternative (regenerate an Isaac-only hero
    image and reference it from the README); we took the smaller correct
    path (removal) since the Mermaid diagrams already cover the visuals.

Closes #113.

Project board: please move #113 to "In review" (board:
Strands Labs - Robots).

…ds-labs#113)

The root images/ folder was outdated and orphaned. Nothing in the repo
referenced these assets, and they had not been touched since the initial
commit (44b5aab, 2026-02-04) -- predating the Isaac-only re-scope (strands-labs#8),
the README/docs rewrite (strands-labs#72), and the MkDocs site.

Removed:
- images/architecture_overview.png  (pre-re-scope legacy SimEnv design)
- images/how_it_works_flow.png       (superseded by mermaid diagrams)
- images/libero_example.mp4          (old MuJoCo-era demo)

The current architecture/flow visuals are rendered inline as mermaid
diagrams in docs/index.md ("How it works") and docs/architecture.md.
The docs site carries its own current assets under docs/assets/.

Also updated NOTICE: the "Images and Media" stanza listed the three
deleted files. Replaced with the current originals (docs/assets logo +
inline mermaid diagrams) so the attribution stays accurate.
@yinsong1986 yinsong1986 requested a review from cagataycali June 18, 2026 12:47
@cagataycali cagataycali merged commit d796d30 into strands-labs:main Jun 19, 2026
1 of 2 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.

Remove the outdated/orphaned root images/ folder (unreferenced since the initial commit; superseded by mermaid + docs/assets)

2 participants