Content
88%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a lean, highly actionable reference built around real failure modes with explicit validation and error-recovery guidance. Its main weaknesses are duplication between the Gotchas and Troubleshooting tables and a dangling kernel.py reference with no bundle present.
Suggestions
Merge the overlapping rows of the Gotchas and Troubleshooting tables (lfc_mean/IORegistryError/use_gpu appear in both with near-identical fixes) into a single symptom→cause→fix table.
Ship kernel.py in the skill bundle (e.g. scripts/kernel.py) or inline the two helper definitions, since the exec(open("scvi-tools/kernel.py")) path resolves to no file present in the skill directory.
Trim the 14-column vanilla DE listing to just the key absence (no lfc_*/proba_de) and link the full schema to a reference file, keeping the change-mode guidance inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — it documents non-obvious failure modes (vanilla DE default, ArrowStringArray write error, removed use_gpu kwarg) without explaining known concepts. However the Gotchas and Troubleshooting tables repeat the same three errors with near-identical fixes, and the 14-column vanilla DE listing is heavy. Not 5 because this duplication could be trimmed; not 3 because over-explanation is minor relative to the overall lean content. | 4 / 5 |
Actionability | Copy-paste-ready scVI, scANVI, and DE recipes with exact kwargs (setup_anndata(layer="counts", batch_key="batch"), accelerator="gpu", devices=1) and a version-pinned Modal script with concrete CLI invocation. Not 4 because the common cases are covered by fully executable code; the only placeholder ("see the recipe above") is an explicitly justified cross-reference. | 5 / 5 |
Workflow Clarity | Clear sequence (load helpers → prepare/validate counts → train → embed → DE → write) with an explicit validation checkpoint (prepare_scvi_counts, provenance verification before training) and error→fix feedback loops (Troubleshooting table plus the NameError recovery hint). Not 4 because checkpoints are explicit, not merely implied. | 5 / 5 |
Progressive Disclosure | Well-sectioned single file with clear headers, tables, and a Next pointer. However the body references "scvi-tools/kernel.py" and a "remote-compute-modal" skill while no bundle files (references/, scripts/, assets/) exist in this skill's directory, so the primary referenced path cannot be resolved. Not 5 because the kernel.py reference is a navigation gap; not 3 because the inline content is appropriately organized for a single-file skill. | 4 / 5 |
Total | 18 / 20 Passed |