[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-04 Thread Jean-Baptiste Mardelle
https://bugs.kde.org/show_bug.cgi?id=484752

Jean-Baptiste Mardelle  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED
   Version Fixed In||24.02.2

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-03 Thread Benoît Tarrade
https://bugs.kde.org/show_bug.cgi?id=484752

--- Comment #7 from Benoît Tarrade  ---
Hi Jean-Baptiste, 

Nice, I tested with the latest revision of master (pulled from GitLab repo)
with commit hash 8094aed4d (Merge branch 'release/24.02')
and it seems to do the trick, now I can open the very large panoramic images.
So, on Arch it works with a local build.

Thanks for your time on this one !

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-02 Thread Jean-Baptiste Mardelle
https://bugs.kde.org/show_bug.cgi?id=484752

--- Comment #6 from Jean-Baptiste Mardelle  ---
Git commit 36d9d2d76f329185d4daab1f4e7c2e4c6990ddb7 by Jean-Baptiste Mardelle.
Committed on 03/04/2024 at 05:08.
Pushed by mardelle into branch 'release/24.02'.

Increase Qt6 limit for max image size

M  +6-0src/core.cpp

https://invent.kde.org/multimedia/kdenlive/-/commit/36d9d2d76f329185d4daab1f4e7c2e4c6990ddb7

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-02 Thread Jean-Baptiste Mardelle
https://bugs.kde.org/show_bug.cgi?id=484752

Jean-Baptiste Mardelle  changed:

   What|Removed |Added

 Status|NEEDSINFO   |CONFIRMED
 Ever confirmed|0   |1
 Resolution|WAITINGFORINFO  |---

--- Comment #5 from Jean-Baptiste Mardelle  ---
Thanks for your investigation and feedback! You are correct, this bug is caused
by Qt6's new limit on image size. I am working on a fix.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-02 Thread Benoît Tarrade
https://bugs.kde.org/show_bug.cgi?id=484752

--- Comment #4 from Benoît Tarrade  ---
Hi again, very interesting : 
I bubbled up the issue to Arch's kdenlive package maintainer, and he managed to
pinpoint the issue coming from Gwenview.
Here is the link to the bug I raised on Gitlab :
https://gitlab.archlinux.org/archlinux/packaging/packages/kdenlive/-/issues/4
which I duplicated from this one.

Quoting Antonio Rojas : 
> This is just an instance of https://bugs.kde.org/show_bug.cgi?id=482195

And if you look at the ticket, I can see the same logs : 
```
::
/// creatclipsfromlist QList(QUrl("file:///.jpg")) true
"2"
/// createClipFromFile "/.jpg" "2"
=== GOT DROPPED MIME:  "image/jpeg"
/// final xml "\n
/.jpg\n\n"
STARTING LOAD TASK FOR:  4  =  "/.jpg" 

:::
qt.gui.imageio: QImageIOHandler: Rejecting image as it exceeds the current
allocation limit of 256 megabytes   
<- Here is the interesting part
which is common to Gwenview's bug !
qt.gui.imageio: QImageIOHandler: Rejecting image as it exceeds the current
allocation limit of 256 megabytes
### ProjectClip::setproducer #
### ClipController::updateProducer
### ClipController::addmasterproducer FOR:  "4"
== READY FOR TASK DISCARD ON:  4
// WARNING EMPTY CLIP HASH: 
===

SETTING AUDIO DATA IN MONITOR EMPTY!!!
=== GOT THUMB FOR:  -1 x -1
MUTEX LOCK setmodel
MUTEX UNLOCK setmodel
MUTEX LOCK loadEffects COUNT:  0
ACTION:  " Custom job"  =  "custom;"
 COMPARING ACTIONTYPE:  ""  =  ClipType::Video
ACTION:  "écoupage automatique des scènes..."  =  "scenesplit;v"
 COMPARING ACTIONTYPE:  "v"  =  ClipType::Video
ACTION:  ""  =  "stabilize;v"
 COMPARING ACTIONTYPE:  "v"  =  ClipType::Video
ACTION:  "Dupliquer  vidéo avec une modification de vitesse"  = 
"timewarp;av"
 COMPARING ACTIONTYPE:  "av"  =  ClipType::Video
ACTION:  " les tâches de vidéos..."  =  ""
 COMPARING ACTIONTYPE:  ""  =  ClipType::Video
== READY FOR TASK DISCARD ON:  4
STARTING LOAD TASK FOR:  4  =  "qimage:/.jpg" 

:::
### ProjectClip::setproducer #
### ClipController::updateProducer
// replace finished:  "4"  :  qimage:/.jpg
== READY FOR TASK DISCARD ON:  4
// WARNING EMPTY CLIP HASH: 
=== GOT THUMB FOR:  -1 x -1
=== 
```

And I've tried again on a real Arch linux system, and this time the *AppImage
version behaved exactly as the one packaged with pacman !*
I had exactly the same logs as well !

I've added a fake picture so that you can try for yourself.
So far it seems related to Qt6 and default QImageIOHandler behavior where
buffered image (uncompressed raw pixels) exceeds 256MBytes of RAM. I believe
that's the case we are hitting with such big images.
This is documented here on Qt6 documentation :
https://doc.qt.io/qt-6/qimageiohandler.html#allocateImage

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-02 Thread Benoît Tarrade
https://bugs.kde.org/show_bug.cgi?id=484752

--- Comment #3 from Benoît Tarrade  ---
Created attachment 168070
  --> https://bugs.kde.org/attachment.cgi?id=168070=edit
Panoramic image (15936x4320px)

Attached a repro image.
1. Create a new Kdenlive project
2. Import the picture (drag and drop or "Add clip" button)

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-02 Thread Benoît Tarrade
https://bugs.kde.org/show_bug.cgi?id=484752

--- Comment #2 from Benoît Tarrade  ---
Hi Jean-Baptiste, 

As always, you're pretty much spot on !
It works just fine with the AppImage : I tried it from a WSL2 Ubuntu and with
the appropriate dependencies installed panoramic images are displayed and
rendered as they should.
So I believe you're right with the packaging issue and it might be reported to
arch linux maintainers.

By the way, I'm sorry for the poor bug report quality, kde bug form is lacking
the edition functionality and a misclick leads us to writing half-cooked
bugs/comments (...)

I'll double-check that on a real Arch box the AppImage is also behaving as it
should, and for the sake of it I'll try with a local build of Kdenlive and
report back if things are not behaving as expected.

Thanks for your time !

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdenlive] [Bug 484752] [Panoramic Image] regular .jpeg panoramic image is considered as a clip when imported

2024-04-01 Thread Jean-Baptiste Mardelle
https://bugs.kde.org/show_bug.cgi?id=484752

Jean-Baptiste Mardelle  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|REPORTED|NEEDSINFO

--- Comment #1 from Jean-Baptiste Mardelle  ---
This looks like a packaging issue. Could you try with the AppImage version just
to confirm if it works :
https://download.kde.org/stable/kdenlive/24.02/linux/kdenlive-24.02.1-x86_64.AppImage.mirrorlist

If it works, then you have a packaging problem - probably some missing or not
compiled MLT modules. If it also doesn't work with the appimage, please provide
a link or attach an image triggering the problem.

-- 
You are receiving this mail because:
You are watching all bug changes.