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

Failed to decode image: BMFF parsing failed - allow no-strict avif decode #28

Closed

DeviantBadge opened this issue Sep 29, 2023 · 1 comment · Fixed by #57

DeviantBadge commented Sep 29, 2023 •

edited

Loading

Mac Ventura - 13.5
Python - 3.8.18

Hello @fdintino !

If we run following code on attached image:

import pillow_avif
pillow_avif.AvifImagePlugin.AvifImageFile('6963c773e084a08167962a5a05d6.avif')

We will get an error:

Traceback (most recent call last):

  File ~/miniconda3/envs/selenium/lib/python3.8/site-packages/IPython/core/interactiveshell.py:3508 in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)

  Cell In[2], line 1
    pillow_avif.AvifImagePlugin.AvifImageFile('/Users/evgenivorobyev/Downloads/6963c773e084a08167962a5a05d6.avif')

  File ~/miniconda3/envs/selenium/lib/python3.8/site-packages/PIL/ImageFile.py:117 in __init__
    self._open()

  File ~/miniconda3/envs/selenium/lib/python3.8/site-packages/pillow_avif/AvifImagePlugin.py:61 in _open
    self._decoder = _avif.AvifDecoder(

  File <string>
SyntaxError: Failed to decode image: BMFF parsing failed

But if we run console command avifdec with --no-strict option (without it, avifdec also fails with similar error), decoding will finish correctly.

May you add opportunity to turn on --no-strict option?

Image is compressed in zip archive, because github can't handle avif image file:
6963c773e084a08167962a5a05d6.avif.zip

The text was updated successfully, but these errors were encountered:

Charlyo commented May 29, 2024 •

edited

Loading

Same is happening to me. How should we fix it?

fdintino added a commit that referenced this issue


          fix: disable decoder strictness for clap and pixi properties

0be3b37

fixes #28

fdintino mentioned this issue

fix: disable decoder strictness for clap and pixi properties #57

Merged

fdintino added a commit that referenced this issue


          fix: disable decoder strictness for clap and pixi properties

a91a81e

fixes #28

fdintino added a commit that referenced this issue


          fix: disable decoder strictness for clap and pixi properties

d5f53fc

fixes #28

fdintino closed this as completed in

506b7ad

fdintino closed this as completed in #57

github-merge-queue bot referenced this issue in BSStudio/bss-web-file-api


          Update dependency pillow-avif-plugin to v1.4.4 (#136)

62478dc

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [pillow-avif-plugin](https://togithub.com/fdintino/pillow-avif-plugin)
| `1.4.3` -> `1.4.4` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/pillow-avif-plugin/1.4.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/pillow-avif-plugin/1.4.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/pillow-avif-plugin/1.4.3/1.4.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/pillow-avif-plugin/1.4.3/1.4.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fdintino/pillow-avif-plugin (pillow-avif-plugin)</summary>

###
[`v1.4.4`](https://togithub.com/fdintino/pillow-avif-plugin/releases/tag/v1.4.4)

[Compare
Source](https://togithub.com/fdintino/pillow-avif-plugin/compare/v1.4.3...v1.4.4)

#### What's Changed

- chore(ci): bump libavif to
[`e10e6d9`](https://togithub.com/fdintino/pillow-avif-plugin/commit/e10e6d9)-2024-07-01;
fix CI build issues by [@&#8203;fdintino](https://togithub.com/fdintino)
in
[https://github.com/fdintino/pillow-avif-plugin/pull/53](https://togithub.com/fdintino/pillow-avif-plugin/pull/53).
See table below for new versions (all versions are upgrades from the
1.4.3 release).

    |             |            |
    |-------------|------------|
| **libavif** | <b>1.0.3
([e10e6d9](https://togithub.com/AOMediaCodec/libavif/commit/e10e6d98e6d1dbcdd409859a924d1b607a1e06dc))</b>
|
    | **libaom**  | **3.9.1**  |
    | **dav1d**   | **1.4.3**  |
    | **SVT-AV1** | **2.1.1**  |
    | **rav1e**   | **0.7.1**  |

- feat: Allow users to pass `max_threads` to the avif encoder via
`Image.save` by [@&#8203;yit-b](https://togithub.com/yit-b) in
[https://github.com/fdintino/pillow-avif-plugin/pull/54](https://togithub.com/fdintino/pillow-avif-plugin/pull/54),
originally in
[https://github.com/fdintino/pillow-avif-plugin/pull/49](https://togithub.com/fdintino/pillow-avif-plugin/pull/49)

- feat: Let users pass `max_threads` as an argument to
`_avif.AvifDecoder` by [@&#8203;yit-b](https://togithub.com/yit-b) in
[https://github.com/fdintino/pillow-avif-plugin/pull/50](https://togithub.com/fdintino/pillow-avif-plugin/pull/50)

- chore(ci): build SVT-AV1 for aarch64 or arm64 by
[@&#8203;RaphaelVRossi](https://togithub.com/RaphaelVRossi) in
[https://github.com/fdintino/pillow-avif-plugin/pull/38](https://togithub.com/fdintino/pillow-avif-plugin/pull/38)

- fix: keep alpha channel for images with mode P and custom transparency
by [@&#8203;fdintino](https://togithub.com/fdintino) in
[https://github.com/fdintino/pillow-avif-plugin/pull/56](https://togithub.com/fdintino/pillow-avif-plugin/pull/56);
fixes
[https://github.com/fdintino/pillow-avif-plugin/issues/48](https://togithub.com/fdintino/pillow-avif-plugin/issues/48)

- fix: disable decoder strictness for `clap` and `pixi` properties by
[@&#8203;fdintino](https://togithub.com/fdintino) in
[https://github.com/fdintino/pillow-avif-plugin/pull/57](https://togithub.com/fdintino/pillow-avif-plugin/pull/57).
fixes
[https://github.com/fdintino/pillow-avif-plugin/issues/13](https://togithub.com/fdintino/pillow-avif-plugin/issues/13),
fixes
[https://github.com/fdintino/pillow-avif-plugin/issues/28](https://togithub.com/fdintino/pillow-avif-plugin/issues/28)

- fix(ci): lint secrets permission error and macOS GHA runner homebrew
`PATH` bug by [@&#8203;fdintino](https://togithub.com/fdintino) in
[https://github.com/fdintino/pillow-avif-plugin/pull/55](https://togithub.com/fdintino/pillow-avif-plugin/pull/55)

#### New Contributors

- [@&#8203;RaphaelVRossi](https://togithub.com/RaphaelVRossi) made their
first contribution in
[https://github.com/fdintino/pillow-avif-plugin/pull/38](https://togithub.com/fdintino/pillow-avif-plugin/pull/38)

**Full Changelog**:
fdintino/pillow-avif-plugin@v1.4.3...v1.4.4

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/BSStudio/bss-web-file-api).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MjUuMSIsInVwZGF0ZWRJblZlciI6IjM3LjQyNS4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment