62 replies to this topic

[domi-sat]

Danke für eure Rückmeldungen und das ausprobieren.

Ich habe mich ja selbst schon im Code eingelesen.

Kann man die Python Sachen einzeln kompilieren und austauschen oder muss man immer komplett das Image bauen, um zu testen?

 

Was passiert denn eigentlich im Code, wenn man eine zweite Menüebene aufruft?

Wenn ich PiP einschalte und Menü aufrufe gibt es das Problem.

Wenn ich PiP einschalte, Menü aufrufe und dann z. B. auf Timer gehe, ist das Problem weg.

Auch wenn ich mit Exit wieder eine Ebene höher gehe passt es (Hauptbild vollständig im MiniTV).

Könnte man also diese Funktion zum Verkleinern einfach zweimal oder zur Sicherheit gleich dreimal aufrufen?

 

Wie ist denn eigentlich das gewünschte Verhalten?

Soll Hauptbild UND PiP in den MiniTV wandern?

Das habe ich noch nie gesehen, wäre aber auch nicht schlecht.

@littlesat Could you reproduce the bug with one of your receivers? Which devices do you have?

Do I really have "luck" to own to receivers with "difficult" drivers?

[Jork]

This is for the full screen window.... and now you're hiding here the PIP when it is set to some value... that cannot be a solution.

Sounds logic

 

Ich bemerke keinerlei probleme. Wirkt einwandfrei fur GigaBluequad4k.

Edited by Jork, 16 April 2024 - 18:38.

[Jork]

Danke für eure Rückmeldungen und das ausprobieren.

Ich habe mich ja selbst schon im Code eingelesen.

Kann man die Python Sachen einzeln kompilieren und austauschen oder muss man immer komplett das Image bauen, um zu testen?

 

 

Man kann eventuel das bestimmte .py im usr/lib/enigma2/python/screens manuel ersetzen. (Mach erst mal eine reserve kopie vom original).

Dann wird bei einem enigma2 neustart automatisch das .pyc kompiliert und erzeugt in das selbe verzeichnis.

Achte darauf das mit software update die original .py zuruckgesetzt wird.

Edited by Jork, 16 April 2024 - 21:58.

[domi-sat]

@littlesat

Ich habe die PictureInPicture.py von OpenPLi9.0 angepasst, sodass der Workaround erhalten bleibt und der Bug gefixt ist.

Das Problem war dieser Commit vom 25.05.2017: https://github.com/O...b0ae44374759877

Dort wurde auf die bestehende resize Methode umgestellt, die aber auch das Hauptbild verändert.

Jetzt gibt es eine eigene Methode für den Workaround (PiP verstecken), der nur das PiP Bild anpasst.

Die nicht mehr benötigte "doSave" Erweiterung aus dem o.g. Commit habe ich entfernt.

Bei meiner Vu+ Duo4K funktioniert damit alles wie es soll. Auf dem ET9000 habe ich noch nicht getestet, will das aber noch machen.

BTW: Den möglichen Bug bei PiP mit UHD Sendern konnte ich weder vor noch nach meinen Änderungen nicht mehr reproduzieren.

Kannst du das so für die OpenPLi 7, 8, 9 und dev Branches übernehmen?

 

@Jork

Danke für die Info mit den Python Dateien. Ich wusste nicht, dass die immer neu kompiliert werden. Damit konnte ich schnell entwickeln.

Magst du mal mit meinen Änderungen testen? Du musste halt manuell anpassen, weil du ja auf dev bist. Das "doSave" von move und resize habe ich entfernt und eine neue Methode hinzugefügt, die in PipPigMode (nach import skin) aufgerufen wird.

Attached Files

•  PictureInPicture_9.0_fix.py   8.7KB   4 downloads

[Jork]

Soweit läuft das alles richtig auf GBQuad4k.

 

Danke und bleibt merkwürdig das keiner  seit 2017 es bemerkt hat.

Schöne leistung!

Edited by Jork, 17 April 2024 - 16:40.

[littlesat]

Ich habe angst dass es immer noch nicht eine loesung ist fuer alle Boxen und Treiber... 

 

@Domi-Sat kannst due einen diff posten?

[domi-sat]

I think it would be easier if you do it on your own with your tools, but ok, here you go.

As you see I didn't change much and don't use other methods than before but just reverted parts of your commit and fixed the problems.

The new method I added is just there to make clear that you may not use the other "move" method in this specific case.

 

FYI:

You removed the lines "self["video"].instance.move(ePoint(x, y))" and "pip["video"].instance.move(ePoint(x, y))" in your commit and that was correct and important.

With this lines you get other position problems.

Attached Files

•  PictureInPicture_9.0_fix.py.diff   1.83KB   5 downloads

Edited by domi-sat, 17 April 2024 - 19:52.

[littlesat]

This will have other side effects. It is also depended on drivers. So when we resolve it for some boxes it will break for others. This is not easy.
And I still do not see any pictures what exactly is tried to resolved.

Edited by littlesat, 17 April 2024 - 20:35.

[domi-sat]

Please see picture from here: https://forums.openp...dpost&p=1615157

Which side effects do you mean?

Was it no mistake that you refactored the code to use the resize method that also calls "setSizePosMainWindow"? I think it must be a mistake.

The resize does not resize the PiP to hidePosition size if PiP is not "standard" or "cascade". Please review your commit from 2017 and you will see it.

The call of "setSizePosMainWindow" is the problem here. It increases the main window to fullscreen after was minimized to MiniTV what causes the problem on my and @Jork s recievers.

Edited by domi-sat, 17 April 2024 - 20:49.

[littlesat]

Not all boxes support hide the thing and need resize and we have boxes that cannot do size zero. So there is no ‘one’ simple solution here. So when you try to resolve it you have to create a feature option via systeminfo where you can list boxes die method x or method y. Your fix can work for et9k but will break it for others.
7 years ago I did not make a mistake. Due to different response in drivers it might induce side effects.

Edited by littlesat, 17 April 2024 - 20:54.

[domi-sat]

I understood that, but it seems that you didn't understand my code.

I do NOT use the "hide" method and my code does not change the size to 0x0 pixels! The code of https://forums.openp...dpost&p=1618477 still uses your workaround with resizing and moving to 16x16px.

The fix is not just for my ET9000. It is also for my Vu+ Duo4K and the GBQuad4k of @Jork.

[littlesat]

It is also 7 years ago and you remove the doSave stuff… I did it on purpose but I do not have the notes from that time anymore. But it should be there for a reason.

[Jork]

Need to know someone with a box who doesn't support the hide thing. To test and take away your concern. Do you remember what type of box?

[littlesat]

Is difficult to find it afterwards…. But I found this commit with a bigger description when the whole stuff started…
https://github.com/O...93022f724457c09

What I still do not understand is why we could remove the doSave stuff…. As this is actually what you did…

I’m also confused what do you mean with miniTV…. The front display or the pIg you can see with the main video?

Edited by littlesat, 17 April 2024 - 22:05.

[Jork]

So actually... a workaround for a box which at the moment of present time is out of manufacturer support?

[littlesat]

Is it possible for you to do a merge request so we can see what happens. Of course for develop.
I do not have the notes anymore for this stuff more than 7 years ago. I just remember that we previously did stop and restart the pip here, which also has side effects.

Edited by littlesat, 17 April 2024 - 22:16.

[domi-sat]

I removed the "doSave" stuff because it was only used by the two lines you added after "pip = InfoBar.instance.session.pip" during the (over)refactoring of the https://github.com/O...b0ae44374759877 commit.

My IDE (IntelliJ) and manual search confirmed that there is no usage, so I thought it is safe to remove.

I understand the idea and information of https://github.com/O...93022f724457c09 and still use this workaround.

 

Yes, I can do a merge request.

Please be patient as I am going to sleep now

[littlesat]

I just tried it and I could not find issues. It is currently committed in develop... When we do not have side effects we still can consider to backport it into 9.0.... But as Develop is our test environment I prefer to keep it for now there.

 

https://github.com/O...681c4ccf46c1789

[littlesat]

Sorry I needed to reverted it as it was for OpenPLi 9.0... and it does not really fit anymore.... When resolved we should first resolve it in Develop.

[littlesat]

I hope resolved...

 

https://github.com/O...6025b8e56b4fed0

 

Maybe also consider to make that special function that hides it smarter... e.g. when you have overlap for PIP and PIG then ensure the PIP is not located over the PIG... 

Edited by littlesat, 18 April 2024 - 12:55.