Skip to content

update tsci snapshot logic#548

Open
mohan-bee wants to merge 2 commits intotscircuit:mainfrom
mohan-bee:fix/tsci-snapshot
Open

update tsci snapshot logic#548
mohan-bee wants to merge 2 commits intotscircuit:mainfrom
mohan-bee:fix/tsci-snapshot

Conversation

@mohan-bee
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 8, 2026 3:23am

Request Review

- `-u, --update` – write new snapshots to disk.
- `--force-update` – force writing snapshots even when no visual diff is detected.
- `--3d` – also generate 3D preview snapshots.
- `--3d` – generate schematic, PCB, and 3D preview snapshots.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

??? that's not what --3d does

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.

image

@seveibar
i wrote the behaviour,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

that is not what this flag does, this flag adds the 3d preview snapshots

By default, only the PCB and schematic snapshots are produced.

Running without `--update` verifies that the generated output matches the existing snapshots. Differences cause a non-zero exit code.
If `--3d` is specified, the command produces PCB, schematic, and 3D snapshots
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wat

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.

image

that's how the cli working !

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

that is not what this flag does, the flag adds to the existing outputs

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

see comments

@mohan-bee
Copy link
Copy Markdown
Contributor Author

@seveibar ig this is the snapshot behaviour, maybe if this seems abnormal then we need to fix the cli snapshot ig ? need you opinion.

@mohan-bee mohan-bee requested a review from seveibar May 7, 2026 07:02
Comment thread docs/command-line/tsci-snapshot.md Outdated
---

`tsci snapshot` renders your boards to SVG images and compares them with saved snapshots.
`tsci snapshot` renders schematic and PCB SVG images and compares them with saved snapshots. It does not generate 3D snapshots unless you explicitly pass `--3d`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why would this be in the tagline

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

check your work carefully, remember that rejection ratio is measured in the contribution tracker SPR table

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.

2 participants