The gist of it is in http://www.darktable.org/redmine/issues/9616.  The
major issue for me is that for some reason, using the simplified encoding
API functions (which would work with the older version of the library) does
not produce the expected result (I'm still looking into that), and I would
like to resolve that in a way that makes use of existing library methods
(the alternative would be manipulating the input buffer bits manually to
strip out the alpha channel bits and/or change the bit order from ARGB to
RGBA which I think might address the problem...).  The other issue is
supporting EXIF/ICC and XMP data in the format, needs to be done manually
since the webp Container API is currently not bundled with libwebp.  I've
contacted the libwebp maintainer and this should be available sometime
later (but probably after the DT release).  Last, the is the issue with
some of the encoder values which can lead to failure to encode images of
certain sizes.  Right now I've fixed the values to a maximum number which
seems to mostly work, but i dont consider this a good fix...  I will be
getting some time off soon so I can foxcus on refactoring the module to
address most of these, but as it stands I am recommending we pull the
feature for now.


On Thu, Dec 5, 2013 at 10:43 AM, jeremy rosen <jeremy.ro...@enst-bretagne.fr
> wrote:

>
>
>
> On Thu, Dec 5, 2013 at 10:15 AM, Dimitrios Psychogios <
> dpsychog...@gmail.com> wrote:
>
>> Yes, the webp feature as is now, requires libwebp version 0.3 and above.
>>  I will either have to make sure CMake checks the library version or revert
>> the module to a version that compiles with 0.1.  Given that I have several
>> other issues to fix before I consider the webp export feature complete and
>> fit for a major release, I think it makes more sense to leave it out of the
>> release for now.
>>
>>
> could you quickly tell us what the other issues are ?
>
> we are considering if we should just disable it by default or completely
> disable the feature...
>
> if it's just a lib version disabling it by default would allow packagers
> to enable it if it works on their distro, but if there are other issues, it
> might be better to remove it from build entirely...
>
>


-- 
Psychogios Dimitrios,
IT Systems Specialist
------------------------------------------------------------------------------
Sponsored by Intel(R) XDK 
Develop, test and display web and hybrid apps with a single code base.
Download it for free now!
http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
_______________________________________________
darktable-devel mailing list
darktable-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/darktable-devel

Reply via email to