Skip to content

Vendor imghdr#74

Draft
valayDave wants to merge 2 commits intomasterfrom
codex/vendor-imghdr-into-metaflow
Draft

Vendor imghdr#74
valayDave wants to merge 2 commits intomasterfrom
codex/vendor-imghdr-into-metaflow

Conversation

@valayDave
Copy link
Copy Markdown
Owner

Summary

  • vendor the imghdr module under metaflow._vendor
  • use metaflow._vendor.imghdr to parse images in cards

Testing

  • python -m py_compile metaflow/plugins/cards/card_modules/convert_to_native_type.py
  • python -m py_compile metaflow/_vendor/imghdr/__init__.py
  • python3 test/core/run_tests.py --num-parallel 8 --debug --contexts dev-local --graphs single-linear-step --tests BasicArtifactTest (fails: No module named 'requests')

https://chatgpt.com/codex/tasks/task_e_68549c69239483339ddd8ddb6cf76011

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant