predictTfExamples replacement method after Zoltar removal #5657
-
|
@kellen I notice in the PR where you removed the Zoltar dependency (e88e454) you also removed the PredictSCollectionOps class, which contained the predictTfExamples method. This method is vital for a project I'm involved in, and it's removal is preventing us from being able to upgrade to the latest version of Scio. Do you have any recommendations about what to replace this with in our codebase? Thanks! |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments
-
|
+1, this seems like a pretty breaking change to users of tensorflow |
Beta Was this translation helpful? Give feedback.
-
|
Zoltar is unsupported and we have effectively no usage of these methods inside spotify. We have no current plans of supporting a replacement in scio. That said, the DoFn code is relatively straightforward so you can copy-paste it from the diff into your own codebase and depend on a stale zoltar (though we are also upgrading to tensorflow 1.0 so you may additionally need to fork and update zoltar, not totally sure on that point). |
Beta Was this translation helpful? Give feedback.
Zoltar is unsupported and we have effectively no usage of these methods inside spotify. We have no current plans of supporting a replacement in scio.
That said, the DoFn code is relatively straightforward so you can copy-paste it from the diff into your own codebase and depend on a stale zoltar (though we are also upgrading to tensorflow 1.0 so you may additionally need to fork and update zoltar, not totally sure on that point).