Channels
announcements
ask-the-community
auth
conference-talks
contribute
databricks-integration
datahub-flyte
deployment
ecosystem-unionml
engineeringlabs
events
feature-discussions
flyte-bazel
flyte-build
flyte-console
flyte-deployment
flyte-documentation
flyte-github
flyte-on-gcp
flyte-ui-ux
flytekit
flytekit-java
flytelab
great-content
hacktoberfest-2022
helsing-flyte
in-flyte-conversations
integrations
introductions
jobs
konan-integration
linkedin-flyte
random
ray-integration
ray-on-flyte
release
scipy-2022-sprint
show-and-tell
sig-large-models
torch-elastic
workflow-building-ui-proj
writing-w-sfloris
Title
r
Ryan Nazareth
11/13/2022, 2:48 AMPushed some changes to https://github.com/flyteorg/flytekit/pull/1240 and made some comments as im struggling a bit with the design (and getting the tests to pass). ☹️
s
Samhita Alla
11/13/2022, 6:25 AMOh sorry about that! I haven’t taken a look at it yet as I’ve been busy this week. Let me review it the next week and get back to you.
r
Ryan Nazareth
11/17/2022, 3:15 AM@Samhita Alla Just pushed some changes and redesigned the transformers as suggested. However still getting an error in the test , the issue is when executing the task and annotating the return as TFRecordFile or TFRecordDirectory , it thinks its ABCMeta metaclass. Ive added comments in PR.