Skip to content

docs: refresh SimA public asset state#298

Merged
DeliciousBuding merged 1 commit into
mainfrom
research/sima-public-refresh-20260525
May 24, 2026
Merged

docs: refresh SimA public asset state#298
DeliciousBuding merged 1 commit into
mainfrom
research/sima-public-refresh-20260525

Conversation

@DeliciousBuding

Copy link
Copy Markdown
Owner

Summary

  • add a 2026-05-25 metadata refresh to the existing SimA evidence note
  • record that the official repo is still on master commit 97dce4fec6030094fc722557a8b03b6858eab37b
  • confirm GitHub still has no releases/tags and no non-vendor split/checkpoint/score/ROC/manifest artifacts

Decision

  • decision unchanged: official code-public mechanism watch only
  • no GPU release, dataset/model download, or Platform/Runtime admission

Checks

  • git diff --check
  • python -X utf8 scripts/check_public_surface.py
  • python -X utf8 scripts/check_markdown_links.py
  • python -X utf8 scripts/export_admitted_evidence_bundle.py --check
  • python -X utf8 scripts/run_pr_checks.py

Copilot AI review requested due to automatic review settings May 24, 2026 19:32
@DeliciousBuding DeliciousBuding merged commit 8e7eb7c into main May 24, 2026
2 of 3 checks passed
@DeliciousBuding DeliciousBuding deleted the research/sima-public-refresh-20260525 branch May 24, 2026 19:33

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Comment on lines +47 to +49
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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

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.

Suggested change
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

@DeliciousBuding DeliciousBuding removed the request for review from Copilot May 24, 2026 19:55
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.

1 participant