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.

Sign up for GitHub

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

Jump to bottom

FindDescriptors fails when no labels specified in AddDescriptor #84

Closed
luisremis opened this issue Feb 23, 2019 · 2 comments
Closed

FindDescriptors fails when no labels specified in AddDescriptor #84

luisremis opened this issue Feb 23, 2019 · 2 comments
Assignees
@luisremis
Labels
Bug Indicates unexpected or undesired behaviors

Comments

@luisremis
Copy link
Contributor

FindDescriptors will fail when not never specified in any AddDescriptor call.

query: [{'FindDescriptor': {'_ref': 1,
'k_neighbors': 3,
'results': {'blob': True},
'set': 'test_emb_50'}}]
response: [{'FailedCommand': 'Transaction',
'info': 'Failed PMGDTransaction',
'status': -1}]
@luisremis
Copy link
Contributor Author

Will be fixed in #85 by adding a default label for all descriptors.

@luisremis luisremis added the Bug Indicates unexpected or undesired behaviors label Feb 23, 2019
@luisremis
Copy link
Contributor Author

Fixed in #85

@luisremis luisremis changed the title FindDescriptors fails when not labels specified in AddDescriptor FindDescriptors fails when no labels specified in AddDescriptor Apr 1, 2019
cwlacewe added a commit to cwlacewe/vdms that referenced this issue Apr 14, 2023
@Ragaad @cwlacewe
Add generic AddBlob command (IntelLabs#79)
fccb2eb 
* Add generic AddBlob command

* Format CMakeLists.txt

* Fix the AddBlob test and include all other tests

* Fix run_test.sh

* Add the add_blob test and fix the blob reading functions for the client examples

* Fix the run_Test to include all the tests

* Aff FindBlob Test code

* Fix the FindBLob test

* ADD UPDATE BLOB TEST and FINDBLOB TEST

* Update dockerfiles (IntelLabs#81) (IntelLabs#83)

* Update dockerfiles; Checkin dockerfiles now only copy vdms folders; remove maven; centralize dependencies in /dependencies folder

* Correct zlib and tiledb paths

* Remove coverage file check

* Change coverage script path

* Change coverage script path

* cd before coverage script

* Add maven build arg to avoid target failure

* Add blob: Update coverage (IntelLabs#84)

* Modify python port and run coverage w/ same cmd

* move large1.jpg to test_images folder, updated paths to image, remove unused image and video in  images folder

---------

Co-authored-by: Chaunte W. Lacewell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luisremis luisremis

Labels
Bug Indicates unexpected or undesired behaviors
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@luisremis @vishakha041