jina-ai / executors

internal-only
Apache License 2.0
31 stars 12 forks source link

migrate SpacyTextEncoder to Jina 3.2 #328

Closed alexcg1 closed 2 years ago

alexcg1 commented 2 years ago

I don't have much experience migrating Executors, but I followed the migration guide and testing.py seems to return decent results with 3.2 now.

Help!

I don't understand much about traversal paths, so if anyone could check those lines of code I'd be grateful

Goals:

PR feedback: @jina-ai/workstream-hub

alexcg1 commented 2 years ago

Can this be merged?