Move examples from cog-examples to cog

[anish-sahoo]

No description provided.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

This PR moves example models from the external cog-examples repo into the main cog repo and updates the test harness to support base_dir for local models. Good cleanup overall.

I found two actionable issues:

1. HIGH: The managed-weights weights.lock was regenerated from incomplete/stub weights. The parakeet model.safetensors is listed as 135 bytes instead of ~2.5GB.
2. MEDIUM: The hello-train model lost its train_tests entry in the manifest, removing training test coverage even though the example still supports training.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. HIGH — examples/managed-weights/weights.lock was regenerated from incomplete/stub weights. The parakeet model.safetensors is listed as 135 bytes instead of ~2.5GB, and the total size dropped from ~2.5GB to ~1.1MB. Please regenerate from the actual weights or revert this file.

2. MEDIUM — hello-train lost its train_tests entry in tools/test-harness/manifest.yaml even though the example still contains train.py and a train: field in cog.yaml. This removes training test coverage. I've posted a suggestion to restore it.

github run

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. HIGH — examples/managed-weights/weights.lock was regenerated from incomplete/stub weights. The parakeet model.safetensors is listed as 135 bytes instead of ~2.5GB, and the total size dropped from ~2.5GB to ~1.1MB. Please regenerate from the actual weights or revert this file.

2. MEDIUM — hello-train lost its train_tests entry in tools/test-harness/manifest.yaml even though the example still contains train.py and a train: field in cog.yaml. This removes training test coverage. I've posted a suggestion to restore it.

3. MEDIUM — examples/z-image-turbo/run.py uses tempfile.mktemp() which creates a predictable temporary file path (flagged by CodeQL). Replace with tempfile.NamedTemporaryFile(delete=False) for safety.

4. MEDIUM — examples/hello-concurrency/run.py has an empty except FileNotFoundError: pass block. Add a logging statement to document the intent.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. HIGH — examples/managed-weights/weights.lock was regenerated from incomplete/stub weights. The parakeet model.safetensors is listed as 135 bytes instead of ~2.5GB, and the total size dropped from ~2.5GB to ~1.1MB. Please regenerate from the actual weights or revert this file.

2. MEDIUM — hello-train lost its train_tests entry in tools/test-harness/manifest.yaml even though the example still contains train.py and a train: field in cog.yaml. This removes training test coverage. I've posted a suggestion to restore it.

3. MEDIUM — examples/z-image-turbo/run.py uses tempfile.mktemp() which creates a predictable temporary file path (flagged by CodeQL). Replace with tempfile.NamedTemporaryFile(delete=False) for safety.

4. MEDIUM — examples/hello-concurrency/run.py has an empty except FileNotFoundError: pass block. Add a logging statement to document the intent.

github run

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. MEDIUM — examples/blur/run.py returns the built-in input function instead of the image parameter when blur == 0. This is a logic bug.
2. MEDIUM — hello-train is still missing its train_tests entry in tools/test-harness/manifest.yaml even though the example contains train.py and a train: field in cog.yaml. This removes training test coverage.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. MEDIUM — examples/blur/run.py returns the built-in input function instead of the image parameter when blur == 0. This is a logic bug.
2. MEDIUM — hello-train is still missing its train_tests entry in tools/test-harness/manifest.yaml even though the example contains train.py and a train: field in cog.yaml. This removes training test coverage.

Both issues have suggestions posted as review comments.

github run

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. MEDIUM — hello-train is still missing its train_tests entry in tools/test-harness/manifest.yaml even though the example contains train.py and a train: field in cog.yaml. The test harness still fully supports train_tests (see manifest.go and runner.go), and there is no visible replacement mechanism or deprecation marker in the codebase. Removing the only training test without a documented alternative creates a coverage gap.

[ask-bonk]

hello-train still supports training (train.py + train: in cog.yaml) but the train_tests coverage was removed. The test harness still processes train_tests, and there is no deprecation comment or replacement mechanism visible in the code. Please either restore the test or, if train_tests is truly deprecated, add a deprecation marker to the manifest schema and runner.

Suggested change

	gpu: false
	gpu: false
	train_tests:
	- description: "train produces weights file"
	inputs:
	prefix: "custom"
	expect:
	type: not_empty
	tests:

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

Summary: Moves example models from cog-examples into the main repo and updates the test harness to support base_dir for local models.

Issues found:

1. MEDIUM — hello-train is still missing its train_tests entry in tools/test-harness/manifest.yaml even though the example contains train.py and a train: field in cog.yaml. The test harness still fully supports train_tests (see manifest.go and runner.go), and there is no visible replacement mechanism or deprecation marker in the codebase. Removing the only training test without a documented alternative creates a coverage gap.

Posted as review comment on PR #3055.

github run