Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please provide a requirements.txt for succussfully running the examples. #215

Closed
fishfree opened this issue Jul 4, 2023 · 4 comments
Closed

Comments

@fishfree
Copy link

fishfree commented Jul 4, 2023

Many thanks!

@junjiejiangjjj
Copy link
Contributor

We have released Towhee 1.1.1, it uses the correct version of pydantic

@595972434
Copy link

595972434 commented Jul 6, 2023

I am using towhee=1.1.1, the example of Reverse Image Search still has issue.
I tried this notebook, everything works well except the step of # Display search results with images, no need for implementation

Here is the error details. I think it might be related to pydantic. I have checked the version of it, I do use pydantic =1.10.10

Cell In[36], line 13
      9         imgs.append(Image(cv2.imread(p), 'BGR'))
     10     return imgs
     12 p_search_img = (
---> 13     p_search_pre.map('pred', 'pred images', read_images)
     14                 .output('img', 'pred images')
     15 )
     16 DataCollection(p_search_img('test/goldfish/*.JPEG')).show()

File [~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/towhee/runtime/pipeline.py:123](https://file+.vscode-resource.vscode-cdn.net/Users/REA_bin.wang/Documents/self-project/milvus/~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/towhee/runtime/pipeline.py:123), in Pipeline.map(self, input_schema, output_schema, fn, config)
    104 def map(self, input_schema, output_schema, fn, config=None) -> 'Pipeline':
    105     """
    106     One to one map of function on inputs.
    107 
   (...)
    121         [1, 2]
    122     """
--> 123     output_schema = self._check_schema(output_schema)
    124     input_schema = self._check_schema(input_schema)
    126     uid = uuid.uuid4().hex

File [~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/towhee/runtime/pipeline.py:517](https://file+.vscode-resource.vscode-cdn.net/Users/REA_bin.wang/Documents/self-project/milvus/~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/towhee/runtime/pipeline.py:517), in Pipeline._check_schema(schema)
    515 @staticmethod
...
File [~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/pydantic/main.py:341](https://file+.vscode-resource.vscode-cdn.net/Users/REA_bin.wang/Documents/self-project/milvus/~/Documents/self-project/milvus/venv/lib/python3.10/site-packages/pydantic/main.py:341), in pydantic.main.BaseModel.__init__()

ValidationError: 1 validation error for TupleForm
schema_data -> 0
  string does not match regex "^[a-z][a-z0-9_]*$" (type=value_error.str.regex; pattern=^[a-z][a-z0-9_]*$)

@595972434
Copy link

595972434 commented Jul 6, 2023

Sorry for raising this issue, I am a beginner of towhee. It seem the previous error is caused by those lines

p_search_img = (
    p_search_pre.map('pred', 'pred images', read_images)
                .output('img', 'pred images')

the schema does not support space, replace pred images with pred_images could solve this issue

@jaelgu
Copy link
Collaborator

jaelgu commented Aug 1, 2023

Sorry for raising this issue, I am a beginner of towhee. It seem the previous error is caused by those lines

p_search_img = (
    p_search_pre.map('pred', 'pred images', read_images)
                .output('img', 'pred images')

the schema does not support space, replace pred images with pred_images could solve this issue

Fixed by #220

@jaelgu jaelgu closed this as completed Aug 1, 2023
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

No branches or pull requests

4 participants