Skip to content

[examples] Introduce new examples#719

Merged
mhs4670go merged 3 commits into
Samsung:mainfrom
mhs4670go:ref1
May 21, 2026
Merged

[examples] Introduce new examples#719
mhs4670go merged 3 commits into
Samsung:mainfrom
mhs4670go:ref1

Conversation

@mhs4670go
Copy link
Copy Markdown
Contributor

This commit introduces new examples.

Related: #716
Draft: #717

TICO-DCO-1.0-Signed-off-by: seongwoo mhs4670go@naver.com

mhs4670go added 2 commits May 20, 2026 13:30
This commit introduces new examples.

TICO-DCO-1.0-Signed-off-by: seongwoo <mhs4670go@naver.com>
Examples:

```text
llama_gptq_ptq.yaml
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.

@mhs4670go
llama_gptq_ptq.yaml is planned to be landed in some next PR?


```bash
python -m tico.quantization.examples.quantize \
--config tico/quantization/examples/configs/llama_gptq_ptq.yaml \
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.

M.b llama_ptq_only.yaml would be more safe ?

```

```bash
python -m tico.quantization.examples.inspect \
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.

This PR does not contain inspect folder 😢

```

```bash
python -m tico.quantization.examples.inspect \
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.

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.

Please refer to the draft when there's codes that are not included in this PR. I just split the draft into few files. But, it was hard to split codes in a single file. It would have benn too burden.


```bash
python -m tico.quantization.examples.quantize \
--config tico/quantization/examples/configs/llama_ptq_only.yaml \
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.

👍 +1


output_dir = cfg.get("export", {}).get("output_dir")
if output_dir:
save_effective_config(Path(output_dir) / "effective_config.yaml", cfg)
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.

👍

def run(self, ctx: RecipeContext, stage_cfg: Mapping[str, Any]) -> RecipeContext:
payload = stage_payload(stage_cfg)
# Build the algorithm config from payload.
# config = AWQConfig(**payload)
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.

@mhs4670go
Is it commented on purpose?

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.

I'll apply this in the next PR.

norm_weight_dtype: int16
strict_wrap: true

evaluation:
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.

@mhs4670go
No way to evaluate original model to assess model degradation?

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.

I'll add this feature.

Copy link
Copy Markdown
Contributor

@stamalakhov stamalakhov left a comment

Choose a reason for hiding this comment

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

LGTM!

VLM example:

```yaml
calibration:
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.

Could you clarify, please, do we plan to use mixed calibration datasets in such recipe configs?

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.

I'll add this feature, too. Thank you.

Copy link
Copy Markdown
Contributor

@Torrero Torrero left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@Torrero
Copy link
Copy Markdown
Contributor

Torrero commented May 20, 2026

We can proceed this PR without @dvsav approval, he is out of the office

@Torrero Torrero removed the request for review from dvsav May 20, 2026 17:07
@mhs4670go mhs4670go merged commit 6183135 into Samsung:main May 21, 2026
7 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.

3 participants