magwyz / pastec

Image recognition open source index and search engine
http://pastec.io
GNU Lesser General Public License v3.0
620 stars 175 forks source link

Limitations on the image ID #24

Closed Sparragus closed 8 years ago

Sparragus commented 8 years ago

I'm trying to add an image to pastec using PUT: /index/images/<id>. The id is a string representation of a very big int. (i.e. '624257033651527680'). However, when I go look up that id later on, I can't find it. Perhaps this is due to some restrictions on the id. If not, then maybe we have a bug.

If there are any limitations on the id, we should specify it on the API. I'd be happy to send in a pull request for this matter after discussing the issue.

Sparragus commented 8 years ago

I'm guessing the limitation is a u_int32_t as seen on requesthandler.cpp. Is this right?

magwyz commented 8 years ago

Yes it is right. It is written at the beginning of the HTTP API section of the documentation: http://pastec.io/doc "Each image has an associated id that is a 32 bit unsigned integer." Maybe it should be specified for each concerned API call.

Sparragus commented 8 years ago

Oh whoa. I should've seen that. Sorry for the noise and thank you for clarifying that!