Skip to content

[data] fix: log skipped VLM dataloader samples as warnings - #62

Merged
kaimo455 merged 1 commit into
masterfrom
codex/vlm-dataloader-warning
May 22, 2026
Merged

[data] fix: log skipped VLM dataloader samples as warnings#62
kaimo455 merged 1 commit into
masterfrom
codex/vlm-dataloader-warning

Conversation

@kaimo455

Copy link
Copy Markdown
Collaborator

Summary

  • replace stderr printing and traceback dumping in the VLM dataloader skip handler with logging.warning
  • keep Energon sample-skip behavior while producing concise warning logs with sample key and exception type

Adapted from internal iCode review 120857904 patchset 1.

Verification

  • final GitHub blob matches iCode patchset 1 for loongforge/data/multimodal/base/task_encoder.py
  • git diff --check origin/master..HEAD
  • python -m compileall -q loongforge/data/multimodal/base/task_encoder.py

…r samples as warnings

Change-Id: Ibfa46a6e01a8bbdb44d1b48eaeb5248b46ef0939
(cherry picked from commit ab58fb7)
@kaimo455 kaimo455 self-assigned this May 22, 2026
@kaimo455
kaimo455 merged commit 86f69f3 into master May 22, 2026
5 checks passed
@kaimo455
kaimo455 deleted the codex/vlm-dataloader-warning branch May 22, 2026 05:53
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.

1 participant