View Issue Details

IDProjectCategoryView StatusLast Update
0006972MMW v4Burning / Disc Handlingpublic2022-04-29 01:08
Reporterrusty Assigned To 
PriorityurgentSeveritytweakReproducibilityalways
Status closedResolutionfixed 
Product Version4.0 
Target Version4.0Fixed in Version4.0 
Summary0006972: CD Configuration inacessible from the Rip CD dialog
DescriptionCertain config items e.g. Secure Rip, On the fly encoding that are applicable to CD Ripping aren't configurable via the CD Ripping dialog.

Moreover, it's strange that Secure Read would be applied to regular CD reading--shouldn't it only apply to ripping?

The issue is all the more relevant because the Accurate Rip dialog allows for the detection of bad rips, but the current workflow doesn't allow the user to change settings on rerip, after such an error has been detected.

Possible approach:
Move checkboxes for On the fly encoding and secure read to the Rip dialog.

To discuss.
Tagstodoc-KB
Fixed in build1344

Relationships

related to 0000243 closedLudek 'Secure' Ripping and New CD/DVD Playback engine 
child of 0006005 closedLudek AccurateRip for verification of disc rips 

Activities

Ludek

2010-12-16 16:07

developer   ~0021870

Last edited: 2010-12-16 16:18

Yes, the UI is now quite problematic

As you indicated [Secure Read] looks like it can be set also for CD playing althouh it is in fact applied only when ripping, so I would suggest:

1. Move
Read Type: [Standard Read^]
to the Rip dialog, but rename it to:
Rip Type: [Standard Read^]

2. eliminate
Read frames: [27]
Overlap frames: [3]
Check frames: [1]
[x] On the fly encoding
because these 4 controls seems to be little bit old-schooled and used very rarely.

3. eliminate 'CD/DVD/BD Devices' page from options entirely, because now only
[x] Read CD-Text
is the only remaining control

4. Now, the problem is:
Where to move '[x] Read CD-Text' ?
Maybe as toggle to the context menu for particular drive, e.g. bellow 'Get Album Info from Freedb', but this doesn't seem to be ideal, any better idea? Eliminating also and leaving this option only in MediaMonkey.ini?

rusty

2010-12-16 18:31

administrator   ~0021871

1. OK
2. I think that 'On the fly encoding' might be worth keeping for some users.
3. We can probably eliminate Read-CD-Text as an option and always have it enabled. If CD text exists and no other metadata is available, then CD text should be used. If CD text exists, and other metadata is available, then CD Text metadata can be presented in the same dialog as other metadata. See the attached graphic.

Summary: the rip CD dialog would have the following options:
Rip Type: __________^
[ ] Verify ripped tracks
[x] On the fly encoding
[ ] Eject CD after ripping
[ ] Level Track Volume

rusty

2010-12-16 18:31

administrator   ~0021872

Reminder sent to: jiri

fyi...please review.

jiri

2010-12-17 11:07

administrator   ~0021897

I think that this kind of simplification would be useful. Just to clarify possible needs for the mentioned (to be removed) options:
a. Read frames, Overlap frames, Check frames - this still might be useful for some types of ripping, particularly for older drive models?
b. On the fly encoding - again, probably useful for older drives/slower machines when this might prevent ripping problems.
c. Read CD-Text - the option to disable is mainly present because it was known problem for some drives that their buggy firmware froze when this option was used.

It seems that all a,b&c are useful only for very old, hardly used anymore setups. So, I think it's safe to remove them from UI, but I'd keep them stored in config files, so that they can be modified by user manually. We would just need a short KB article for this.

Ludek

2010-12-17 13:28

developer   ~0021903

Last edited: 2010-12-17 13:30

Fixed in build 1336

Note that the old a,b&c options are available in registry:
HKCU\Software\MediaMonkey\CDs\

rusty

2010-12-19 06:44

administrator   ~0021920

Tested 1336--looks good.

Is it possible to have these items configurable via config files (rather than the registry)? It's reasonable to document how to make changes to a config file, but not really how to modify the registry.

Also, I think that on-the-fly encoding should be configurable in the UI. Some users will really want to disable that.

Ludek

2011-01-11 17:02

developer   ~0022302

Fixed in build 1344

1) Added the '[x] On the fly encoding' checkbox
2) Moved CD configuration from registry to MM.INI file, now the drives are stored like:

[CD Drive D]
ReadSectors=27
ReadType=0
JitterOverlap=3
JitterCheck=1
ReadCDText=1
ConvertOnFly=1

peke

2011-02-12 22:45

developer   ~0023030

Verified 1348