docs: refresh SimA public asset state#298
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the SimA artifact gate documentation with a 2026-05-25 refresh, noting that the repository still lacks public releases, tags, and functional links for datasets or checkpoints. Review feedback identifies an inconsistency in the summary of the repository scan, suggesting the inclusion of missing items like notebooks and licenses to align with the detailed evidence table.
| A non-vendor artifact-shaped tree scan excluding `diffusers/`, | ||
| `guided-diffusion/`, `latent-diffusion/`, and `taming-transformers/` found only | ||
| editor config, figures, and `scripts.sh`; it did not find committed public |
There was a problem hiding this comment.
The summary of the tree scan findings in this new section is inconsistent with the detailed evidence table later in the document. Line 63 explicitly mentions that the repository contains DDPM/ code, notebooks, the README, and the license, all of which are omitted from the "found only" list here. This discrepancy could mislead readers about the actual contents of the repository and the scope of the refresh.
| A non-vendor artifact-shaped tree scan excluding `diffusers/`, | |
| `guided-diffusion/`, `latent-diffusion/`, and `taming-transformers/` found only | |
| editor config, figures, and `scripts.sh`; it did not find committed public | |
| A non-vendor tree scan excluding diffusers/, | |
| guided-diffusion/, latent-diffusion/, and taming-transformers/ found only | |
| code, notebooks, figures, scripts.sh, README, license, and editor config; it did not find committed public |
Summary
mastercommit97dce4fec6030094fc722557a8b03b6858eab37bDecision
Checks
git diff --checkpython -X utf8 scripts/check_public_surface.pypython -X utf8 scripts/check_markdown_links.pypython -X utf8 scripts/export_admitted_evidence_bundle.py --checkpython -X utf8 scripts/run_pr_checks.py