Rockbox Technical Forums

Support and General Use => Theming and Appearance Customization => Topic started by: Frankenpod on December 25, 2022, 07:12:29 AM

Title: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 25, 2022, 07:12:29 AM
Noticed that some themes of mine (and also the theme "redbox") have started falling back to the failsafe theme, when used with recent dev versions of RB.  Just checked and they still seem to be working on ipods that have older versions of RB on them (e.g. they work with version dated 210409 but not with 221221).  They seem to work with the simulator also.

(Heh, though the fact that nobody else has complained suggests nobody but me uses them)

I might be barking up the wrong tree (tedious to work it out as the themes still work on the simulators - presumably because they use older versions of RB), but as far as I can narrow it down it seems to be any theme that has a bar-chart for the EQ settings in it.

i.e. the problem seems to be with lines like this:
%St(0,0,-,-,image,eqbar.bmp,vertical,setting,eq band 1 gain)
 
Seems as if "eq band 1 gain" is no longer recognised as a valid setting value.

Using "eq peak filter 1" instead doesn't crash the theme, but returns three numbers, the third of which I think is the gain, but it doesn't seem possible to retrieve the gain on its own now.

Has something changed in how these values are named, or perhaps the scope of where they can be accessed?  What does "eq band N gain" need to be changed to for this to work again?
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: amachronic on December 26, 2022, 07:13:50 AM
It likely broke after those EQ settings were removed by this commit (https://git.rockbox.org/cgit/rockbox.git/commit/?id=6322e66219baba3fea7f4ec3cc622674f2f340f3). I'll just put the "deprecated" settings back in - that seems like the simplest solution and won't require changing any themes.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 26, 2022, 10:58:53 AM
If you think that's the best approach, by all means go with it.  (I think all but one of the themes that have the problem are mine, as it happens)

  Can't say I really know anything about the underlying issue (were there previously two ways, internally, of referring to the same variables?).
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 07, 2023, 10:47:54 AM
I don't suppose anyone is motivated to modify RB code so as to allow these themes to work again?

That is, somehow allow the eq band settings to be accessed by the theme engine again?  Even if it means changing the variable names or something, such that the themes would need amending for new syntax?  At the moment it seems those settings are now entirely inaccessible from the theme engine.

Just hoping some dev might be prepared to at least partly undo the change that broke this feature.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Bilgus on December 08, 2023, 06:54:28 AM
Did the fix by amachronic not work or did he forget?
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: rockbox_dev123 on December 08, 2023, 10:23:01 AM
Did the fix by amachronic not work or did he forget?

https://git.rockbox.org/cgit/rockbox.git/tree/apps/settings_list.c?h=refs/heads/master

A search for "eq band 0 cutoff" returns nothing.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 08, 2023, 10:29:14 AM
Did the fix by amachronic not work or did he forget?

I just noticed it still doesn't work on the most recent builds.

  I wasn't clear from the reply if amachronic was explicitly saying they would apply a fix or if he was just saying that that would be the way to fix it at some point (if someone was prepared to do it).  I didn't want to press the point too much, as it seems as if I'm the only one who is bothered about the issue!
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: rockbox_dev123 on December 08, 2023, 11:36:49 AM
Did the fix by amachronic not work or did he forget?

I just noticed it still doesn't work on the most recent builds.

  I wasn't clear from the reply if amachronic was explicitly saying they would apply a fix or if he was just saying that that would be the way to fix it at some point (if someone was prepared to do it).  I didn't want to press the point too much, as it seems as if I'm the only one who is bothered about the issue!

He's saying it would be the simplest thing to do to just re-add the removed settings options. This doesn't appear to have been done to master because the themes are broken still.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: speachy on December 08, 2023, 08:59:44 PM
I don't suppose anyone is motivated to modify RB code so as to allow these themes to work again?

I'm not personally in favor of bringing the old settings back; instead the theme engine should be fixed and the theme altered as needed. 

The fundamental issue here is that the EQ settings are a triplet, and that the skin engine doesn't know how to handle this.

I just hacked something together that _should_ do the trick, but it may be the wrong approach (and I don't know if it's possible to do this in a backwards-compatible manner)   To use it the theme will need to be tweaked.  See:

   https://gerrit.rockbox.org/r/c/rockbox/+/5532

Be warned:  I can promise that it compiles, but that's the extent of the testing it's received.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 09, 2023, 09:26:37 AM
Thanks very much.

  Can't say I understand the technicalities of it all (how did it work previously, if the parameters being an array is something the theme engine can't handle?  Did it just work entirely by accident before?).

  Will try modifying some themes to the new syntax - but does this mean I'll need to compile a patched version of RB first?

[Edit] I really struggle to do that - I did have a working development environment running at one point, but then my unix machine had a lot of hardware problems, and when I finally got it working again and tried again I could never again manage to get RB to build successfully, there always seemed to be some glitch or other.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Bilgus on December 10, 2023, 06:34:21 AM
Which player do you need us to compile for you iPod classic?
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 10, 2023, 08:40:03 AM
ipod 6g.  Or ipod video, but ideally the former.
Title: Re: Some 320x240 themes seem to have stopped working on recent dev versions
Post by: Frankenpod on December 21, 2023, 02:11:30 PM
I've updated most of the themes affected to use the new syntax, but still not sure how to actually test the new versions.

(Also can't update any already-uploaded themes any more as I managed to get locked out of the email address they are associated with!)