Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search




Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Rockbox Ports are now being developed for various digital audio players!

+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Recording
| | |-+  Recording Screen conversion to ViewPorts
« previous next »
  • Print
Pages: [1] 2

Author Topic: Recording Screen conversion to ViewPorts  (Read 7108 times)

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Recording Screen conversion to ViewPorts
« on: July 24, 2008, 05:01:20 PM »
There is a fully functional converted recscreen in the tracker here (FS #9208), and I wouldn't mind if it would get some testing on various targets before it goes in.

Also let me know what you think of it. At this moment it will revert to sysfont if the whole screen doesn fit anymore, but this could be done incrementally (ie first change the list to sysfont)

The purists will be happy to hear this patch shaves 1760 bytes off the binary for h300 :)
Logged

Offline JdGordon

  • Global Moderator
  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Recording Screen conversion to ViewPorts
« Reply #1 on: July 24, 2008, 07:26:42 PM »
1) arnt you one of those purists... :D

2) (i havnt looked at the patch yet..) bassically the whole point of viewports was in the hope that screens wouldnt need to revert to the sysfont (which also causes the problem where LANG_SYSFONT_* is still needed).

OK, I've just had a quick look at the code and I have some suggestions.
The peakmeter should go in a seperate viewport and use the whole thing so its the easiest thing to resize.
The list needs a minimum of one line so the whole screen then needs min 5 (3+1+1). and if that still is too many, have some fun with the top 3 lines... only show one line, but alternate the text every HZ/2 or something...?
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #2 on: July 25, 2008, 02:32:59 AM »
Quote from: JdGordon on July 24, 2008, 07:26:42 PM
1) arnt you one of those purists... :D
Which is why I wrote that :P

Quote from: JdGordon on July 24, 2008, 07:26:42 PM
2) (i havnt looked at the patch yet..) bassically the whole point of viewports was in the hope that screens wouldnt need to revert to the sysfont (which also causes the problem where LANG_SYSFONT_* is still needed).

OK, I've just had a quick look at the code and I have some suggestions.
The peakmeter should go in a seperate viewport and use the whole thing so its the easiest thing to resize.
The list needs a minimum of one line so the whole screen then needs min 5 (3+1+1). and if that still is too many, have some fun with the top 3 lines... only show one line, but alternate the text every HZ/2 or something...?
The problem is, the user is able to pick a font that hardly fits two lines on the display. What then?
I agree that things can be tweaked a bit more, but fallback to sysfont seemed like a nice 'emergency' remedy.
I hate cludges like alternating text. The changing behavior is hard to explain ('my display suddenly looks different') and document because it depends on the font size.

Anyways, I first want to commit this as it stands now, and then start tweaking how the top part is drawn. Too much change in one go is dangerous. And it almost looks the same now as it was before.
Logged

Offline z-man

  • Member
  • *
  • Posts: 9
    • Z-Man's home
Re: Recording Screen conversion to ViewPorts
« Reply #3 on: July 25, 2008, 03:04:06 AM »
The idea of having kind of a "master" gain is an improvement. But I have my concerns about the usability of the current implementation. Let's say, I'm doing a live recording, but for some reason there's a need to have more gain on the LEFT channel. No problem to get it done with the left/right faders. Okay. What will happen next: The band becomes louder than expected. That is: with both channels still unbalanced, I have to fade down the master gain. And currently, there's a good chance to change the left/right balance and fade the LEFT channel by mistake when the RIGHT channel is already at 0.
So, with 3 gains I would prefer to have a master gain, that has no impact on the left/right balance. Or even more straightforeward: Just one master gain, and kind of a "recording balance" slider with  0 dB (=balanced) in the middle.
Logged

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #4 on: July 26, 2008, 04:52:31 PM »
z-man: regarding your findings:
- the X5 compile error is fixed in my latest patch
-  CLIP counter does work

there seems to be something odd with changing gain though, investigating that...
Logged

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #5 on: July 26, 2008, 06:35:32 PM »
New patch is up, all known issues fixed. Left to do are LANG conversio and maybe fancy peakmeters
Logged

Offline LurkAzusa

  • Member
  • *
  • Posts: 40
Re: Recording Screen conversion to ViewPorts
« Reply #6 on: July 26, 2008, 08:27:35 PM »
Thank you, Petur.  It is very readable on the H120 and H340.  It's great to have the ability to choose a very large font and still scroll to the information.  Appears to work well. 
Will the screen design be the same as wps or will there be a specific wpr function?
Logged

Offline JdGordon

  • Global Moderator
  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Recording Screen conversion to ViewPorts
« Reply #7 on: July 26, 2008, 08:33:29 PM »
Quote from: LurkAzusa on July 26, 2008, 08:27:35 PM
Will the screen design be the same as wps or will there be a specific wpr function?

the screen is hardcoded, at least for the time being there isnt going to be any wps-like ability for the rec screen
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline LurkAzusa

  • Member
  • *
  • Posts: 40
Re: Recording Screen conversion to ViewPorts
« Reply #8 on: July 26, 2008, 09:03:18 PM »
Will there be the ability to add a large icon representing the current status of recording, pause, stopped?

Or changing the field identifier to a shorter abbreviation, such as file, instead of filename or PR or PreRec instead of Pre-Recording?  This would allow larger fonts without cutting off information.
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Recording Screen conversion to ViewPorts
« Reply #9 on: July 26, 2008, 09:08:32 PM »
Changing text should be as simple as editing your language file.
Logged

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #10 on: July 27, 2008, 02:16:39 AM »
The plan is to first get this change in, and then we can start tweaking it and in the future even have an WRS gui system.

Things I'd like to tweak:
- get the listbox to scroll the selected line, to show more of the filename
- fancy peakmeters (now that the have become that big)
- clear recording state indication, maybe software drawn and thus scaleable.
Logged

Offline LurkAzusa

  • Member
  • *
  • Posts: 40
Re: Recording Screen conversion to ViewPorts
« Reply #11 on: July 27, 2008, 02:03:32 PM »
I've tried it on the Ipod 4G Greyscale and Ipod 5G 60. 
Seems to be good.

 The volume and gain settings go negative with a right press, which seems counter intuitive, but that is also in the original.


Logged

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #12 on: July 28, 2008, 04:49:14 AM »
Quote from: JdGordon on July 24, 2008, 07:26:42 PM
The peakmeter should go in a seperate viewport and use the whole thing so its the easiest thing to resize.
I have given this some thought...

Are suggesting this in the light of a future WRS implementation? If not, I do not see any advantage in giving it a separate VP.

For the moment, keeping it like I have gives the possibility to add some stuff to left and right of the peakmeter. I would also like to implement a vertical PM so it can be put on the right of the screen and give us another extra line. But in its own VP? Nah...
Logged

Offline JdGordon

  • Global Moderator
  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Recording Screen conversion to ViewPorts
« Reply #13 on: July 28, 2008, 07:33:57 AM »
well, if its in its own vp the drawing funciotn for it is dead simple, (x,y,width,height) is just (0,0,vp->width, vp->height)
I have no other reason for that thoguh
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline petur

  • Developer
  • Member
  • *
  • Posts: 769
  • wtb: time
Re: Recording Screen conversion to ViewPorts
« Reply #14 on: July 28, 2008, 09:53:27 AM »
Quote from: JdGordon on July 28, 2008, 07:33:57 AM
well, if its in its own vp the drawing funciotn for it is dead simple, (x,y,width,height) is just (0,0,vp->width, vp->height)
I have no other reason for that thoguh
I'd rather have code that can position the PM anywhere I like... the PM still needs to get x and y because it is also used elsewhere.
Logged

  • Print
Pages: [1] 2
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Recording
| | |-+  Recording Screen conversion to ViewPorts
 

  • SMF 2.0.6 | SMF © 2013, Simple Machines
  • XHTML
  • RSS
  • WAP2

Page created in 0.138 seconds with 68 queries.