Skip to content

fetch: 'builder error' on image-sibling + parquet-only datasets #76

Description

@vxfemboy

Mirrored from Forgejo — originally by @vxfemboy. https://virus.ad/femboy/kibble/issues/84

kibble fetch "builder error" on datasets with image siblings or parquet-only layouts

Some HF datasets fail with download failed for <id>: builder error while others (single .jsonl) succeed. Reproduced building the kitby-v1 corpus:

  • ianncity/GLM-5.2-Conversation — a single dataset.jsonl plus a .webp sibling → builder error (had to direct-download the jsonl).
  • r0b0tlab/qwen3.8-max-distillation-50kparquet shards under data/ (no top-level jsonl) → builder error.
  • SupraLabs/reasoning-corpus-4K-5M-v1 (single jsonl) and greghavens/kimi-k3-... (jsonl) → succeed.

The message text (download failed for {url}: {e} with {url} = the bare dataset id, {e} = builder error) points at a bare id reaching reqwest as a URL on the parquet-convert / fallback path — a RequestBuilder build failure, not an HTTP error. is_data_file already accepts parquet, so the parquet case should resolve via data/ or the parquet-convert branch; something in that path constructs the request from the id instead of a resolve URL.

Impact: common dataset layouts (image sibling; parquet-only) need a manual direct-download workaround.

Fix direction: ensure every fetch path builds a full resolve/main/... (or /parquet) URL, never the bare id; and skip non-data siblings (.webp/.png/...) before the builder step.

Found during kitby-v1 data foundry (P1).

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions