Any plans on storing the image dimensions in the easy_thumbnails models?

[smileychris]

Any plans on storing the image dimensions in the easy_thumbnails models?

[tuttle]

There's in fact yet another way. The thumbnail w/h are the functions of the source-image's w/h plus the thumbnail_options dict.

In case e-t will provide the calculation API for the code it internally uses like

size_tuple = e-t.utils.thumbnail_dimensions(src_size_tuple, thumbnail_options)

then I'd store the source image w/h in my app's model and upon any image retrieval I'd be able to simply provide the w/h for any image together with its CloudFiles URL.

[SmileyChris]

That [calculating thumbnail dimensions based on source dimensions] sounds like a good way... it'd probably mean having to refactor the processors so they can somehow return size information without actually bothering to muck with PIL.

The other solution would be to add the dimension info to the database cache record of the thumbnail images, but then it's making the rows (slightly) larger for everyone, even if they don't care about the dimensions.

[tuttle]

Ouch, it seems the autocrop is a blocker to post-calculation as it needs the image data to decide how to crop... :(

[SmileyChris]

Thinking about it again, I'm pro putting the source dimensions in the cache models. Let's do it.

[alanjds]

Please considere this issue: http://bit.ly/x7mGse

Not storing the image dimensions makes the storage backend to spend effort trying to get if from the true data, which costs a lot more than wasting columns or hitting the database, if using a remote storage backend like Amazon S3.

I am not saying that its a problem of easy-thumbnails or of Django itself, just that this feature can fix it.

[SmileyChris]

I've already said it's a good idea - just haven't got around to implementing it. It's probably one of the last changes to the models before I reset the migrations (which have caused no end of pain for mysql users) and call easy-thumbnails "1.0".

Feel free to implement this (migrations, code, tests & docs) and fire me a pull request.

[SmileyChris]

Done!