Recording Enhancements Pack

[Mmmm]

OK then, as a sort of experiment here's a patch that allows you to change the font in the recording screen..It will show the same font as WPS and filebrowser... :-)

Post the font you think works best.

(it also includes the rest of the pack)

[attachment deleted by admin, too old]

[Tenn Man]

OK then, as a sort of experiment here's a patch that allows you to change the font in the recording screen..It will show the same font as WPS and filebrowser... :-)

Post the font you think works best.

(it also includes the rest of the pack)

Is this included in your rockbox.zip file?

[Mmmm]

Is this included in your rockbox.zip file?

No, but here's a link to a build that includes it...

Build with Changeable fonts in rec screen

[Mmmm]

Right.. If you have tried the patch above with the changeable fonts you may have noticed that this messes up the peakmeter on the remote...
This is because the remote peakmeter shares the coordinates of the main unit peakmeter. So if you use a different size font the peakmeter moves on the remote!

the possible solutions I can think of are
i) Have the same font on the remote as on the main unit.
   But the remote screen is filled with the default font and any larger font wouldn't fit and we would have the same problem!
ii) Pass special coordinates to the peakmeter draw function for the remote.
 Â  But then any targets that don't use a remote would get confused and not compile.
iii) Hardcode the coordinates of the remote peakmeter into the peakmeter code.
   But then if you wanted to use the peakmeter in your rWPS you wouldnt be able to place in the desired location, it would always appear  in the same place.
iv) Make the recording screen customisable like a WPS.
ermm... bit beyond my ability that one.
v) Totally rewrite the rockbox code so it can be done easily.
I don't think so! :-D

So....do you begin to see why you cant have customisable fonts in a non-customisable screen when at least two different sized screens have to be catered for, not to mention all the screens of the numerous targets that rockbox has to be compatible with? :-)

[qpwoei]

OK then, as a sort of experiment here's a patch that allows you to change the font in the recording screen..It will show the same font as WPS and filebrowser... :-)

Post the font you think works best.

(it also includes the rest of the pack)

Hi, first post here

Mmmm, May God Bless You for helping short-sighted people... like myself :-)
My approach: I Browse Themes, select Marquee. The recording screen becomes soooooooo clear and easy to read... big THANK YOU!
As for the remote, yes, The Volume and Gain lines dissapear with the large font, but I don't care much about this because the meters are visible and clip backlight works ok

cheers

[Mmmm]

Oooh , I've just thought of an option (vi) !!

and I think it'll work..

In fact, it would be better than the current (non changeable font) version as it would make the remote peakmeter totally independent of the main unit one! Great! This would also work for rWPS's hmmmm...

I just call the peak meter code twice. Once for main unit and once for remote, passing a boolean to tell the peakmeter code which it is for. This would work for non-remote targets too....
Cor....I'll try implementing it later....

Fingers crossed! :-D

[dropandhop]

Mmmm,

As usual, great idea!

Have any of the devs commented on your work?  What I am getting at is if there some things that you know are stopping it from making it into the cvs?  Cause that would be splendid!

Keep up the great work!
A

[AGI]

Mmmm,

just tried it with Chicago12 and it is what I've dreamed of :-)
I can see what the problem with the remote is. I really hope your idea of a double code will work, fingers crossed here too!

I think you will also like to know that after I loaded the 'Build with Changeable fonts in rec screen' on my h120 I got again the strange different channel gains problem which I got rid of by deleting settings. I am also now having problems playing back files: I get 'codec failure' messages and a lot of HD activity. Not sure, but this could be related to me having played too much with different builds lately :-). I dont have the time to get back to a fresh build now. I'll try to do it tomorrow and I let you know if I get those problems again.

Once more, great work! Thanks a million!

[Mmmm]

Wow...it works...!!

Even though i've just spent about 4 hours unwriting and rewriting,testing, retesting, unwriting again and then starting from scratch...I am now almost bald and there are teeth marks in my desk! :-D

and then I realized that it will only work if you reset the settings first! I had actually got it completed in about 15 mins!
I don't believe it ......................HA..

anyway, give this patch a go.. now the remote stays the same while you can change the font on the main unit..

and here's the build... (patch attached below)
Remember to reset your settings as soon as your player starts
Same link as before: Changable fonts build

Have any of the devs commented on your work?  What I am getting at is if there some things that you know are stopping it from making it into the cvs?  Cause that would be splendid!

well..me and Petur are waiting for the gain settings patch to get committed (hopefully soon) and then we go to the next step.

 I'll have to rewrite the remote patch, or maybe i'll have to try to implement the peakmeter all by itself on the remote (as it doesn't work at all at the mo) and after that, recording on the remote.

Cliplight? don't know, could be put on at any time as it is quite separate from the other code.

It's really just a question of whether I have done it in the right way!
I'm very new to this coding malarky so I'm not really totally confident about whether the way I'm doing things is right or not, even though it works!
and I think that it will be just one slow thing at a time if it does get committed..

AGI: I haven't had any of the codec errors you mentioned... and I thought I had sorted the channel gains prob..(I haven't had it since but you never can tell eh?) :-)
Give this new build a go and see what happens.

[attachment deleted by admin, too old]

[Dick]

Thanks man. This was the last thing I was missing in Rockbox: Just one button to adjust gain. And seperate settings for Mic and line-in is great, too.

[Tenn Man]

Has your rockbox.zip been updated to include this latest work?

[renho3k]

Mmmm, i will donate a bottle of rogain for your sacrifices

*chants*
COM-MIT! COM-MIT! COM-MIT!

[mlind]

I'm still not able to compile myself, so I downoaded the build in Mmmm's sig.

The recording patches are great, but I also noticed:

- The startup screen does not show at all. No big deal, but it probably means something.
- I can't use "browse .wps files" because the first wps in the list immediately gets chosen for use. So to use my own wps I must switch "Show files" to "All" go to .rockbox/wps/ with the normal browser instead.

Something I that might be in the unpatched builds, but I haven't noticed before:
- I listen to a wav (my own recording), turn off the payer, turn it on again, press play. The playback does not resume where I was but starts over from the beginning of the recording.

Finally a microscopical suggestion concerning the Gain Change Patch:
- It's showing (A) or (D) if the gain is changed in the Analog or Digital domain. From 0 to +24 it says (A), but from +24.5 it says (D). Could it be worth the space to have it say (A+D) - as I'm sure is the case.

And! I would like to have the settings for the peak meter duplicated in the recording settings screen.

[Mmmm]

Has your rockbox.zip been updated to include this latest work?

No the changeable fonts isn't included, as I'm not happy with having the same font in WPS as in the recording screen. This means you may have to change fonts to go into recording and then change them again when you play a track! It's just an experiment at the moment.
If I could get a special recording screen font menu then it would be in like a shot, but I was coding all day yesterday to get the peakmeter sorted and need a break! :-)

I have put a link to a build with this in posts #23 and #17. So look there for compiled builds with changeable recording fonts.

Mmmm, i will donate a bottle of rogain for your sacrifices

Ooh...I'm looking forward to it! :-) ......What's rogain? :-D

The startup screen does not show at all. No big deal, but it probably means something.

Yep, mine did that too but when I reinstalled my settings it seemed ok! I don't understand it either. But it doesnt half boot up fast eh? :-D

- I can't use "browse .wps files" because the first wps in the list immediately gets chosen for use. So to use my own wps I must switch "Show files" to "All" go to .rockbox/wps/ with the normal browser instead.

Mine seems fine...no problems at all :-s

- I listen to a wav (my own recording), turn off the payer, turn it on again, press play. The playback does not resume where I was but starts over from the beginning of the recording.

Yep same here, ogg and mp3 ok though. I reckon thats in the normal builds but I haven't tested.

- It's showing (A) or (D) if the gain is changed in the Analog or Digital domain. From 0 to +24 it says (A), but from +24.5 it says (D). Could it be worth the space to have it say (A+D) - as I'm sure is the case.

And! I would like to have the settings for the peak meter duplicated in the recording settings screen.

The A or D is just supposed to say whether the digital gain is being used or not. A if it's not and D if it is. You are right that the digital gain is on top of the analogue but isn't that fairly obvious? .....maybe not... :-)
You are probably right about the peakmeter settings...I think it would be good here too.. but would we then have the same settings in two places?

Blimey I'm writing some long posts these days :-D

[toolmanwill]

WOW, I'm really liking ur patch Mmmm!. I loade3d your already patched build and am loving the bigger font on rec screen. I use 6x13B font size and it makes it fit the whole screen. Peakmeters even get alot larger which is good cause thats prolly the most important thing you'll need to look at while recording. I agree with any others that these changes should be added to cvs for daily builds if possible