View Issue Details

IDProjectCategoryView StatusLast Update
0002373MMW v4Burning / Disc Handlingpublic2006-07-25 19:30
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status resolvedResolutionfixed 
Fixed in Version3.0 
Summary0002373: Enable Auto-Conversion for CD Burning
DescriptionWe should allow the auto-conversion functionality to be used for CD burning in additiona to device synchronization.

-The auto-conversion settings for cd burning should be made after the user selects the device format.
-The auto-conversion settings page should give the user an estimate of much space will be used so that they can add more to the Disc if they want
-The auto-conversion settings should persist independently of the device auto-conversion settings.
TagsNo tags attached.
Fixed in build1002

Relationships

related to 0002107 resolvedLudek Auto-Conversion: Users should be able to define what formats are supported on a device 
related to 0002108 resolvedjiri Auto-conversion: Add Volume Leveling 

Activities

jiri

2006-02-16 21:38

administrator   ~0006894

Assigning to Ludek.
 - Ask me if you have any troubles with this, generally speaking code from device support should be re-usable here.

Ludek

2006-03-23 12:56

developer   ~0007087

Last edited: 2006-03-28 13:34

The Auto-Conversion functionality for burning was implemented in build 962.

There are added a new button and dialog that allow a user to set supported formats by his/her CD player.

There are also several new texts could be changed by Rusty.
i.e.
- When not enough estimated free space for convertion:
'There will be probably not enough free space to convert all tracks. Check the temporary directory free space'
[Cancel] [Retry]

- When Auto-Conversion fails:
'There was a problem during Auto-Conversion'
[Ok]

- Text on showing the new 'Select format' dialog:
'Select formats that your CD player supports.'

jiri

2006-03-23 13:06

administrator   ~0007088

Assigning to Rusty for wording suggestion.

rusty

2006-06-22 03:48

administrator   ~0007360

Last edited: 2006-06-22 04:00

Re. the wording:
-'Enable Auto-Conversion' --> 'Auto-convert audio format'
-'There will be probably not enough free space to convert all tracks. Check the temporary directory free space'
[Cancel] [Retry]
--->'There is insufficient space in the temporary directory to convert all tracks. Do you want to continue?' [[Yes]] [Cancel]

- When Auto-Conversion fails:
'There was a problem during Auto-Conversion'
[Ok]
q) When does this error message occur? Does it occur only once or can it occur multiple times?

- Text on showing the new 'Select format' dialog:
'Select formats that your CD player supports.'
-->'Select formats supported by the the device that will play this CD/DVD.'

Re. Functionality:
1) If the user doesn't have a gold license, then the same popup should appear as when the user tries clicks the Synchronization Auto-Conversion tab.
2) The Auto-Convert option makes more sense on the 'Choose Disc Format' panel, just below the 'Burn .m3u Playlists' option.
3) One aspect of the Auto-convert function on CD/DVD doesn't translate well from portable devices: On a portable device, the device often reports what formats supported, and thus it makes sense to separate the supported devices field (since it's not usually configurable).

However, with a cd/dvd, the device _never_ reports on what formats are supported. Thus it would make more sense to get rid of the 'supported devices' area entirely, and integrate this function into the auto-conversion rules (since right now we're effectively splitting the rule configuration to different portions of the UI).

If this isn't too much work, one possibility is:
Format on PC: any, [any except], selected
 . . . . . . [x] MP3
 . . . . . . [ ] WMA
 . . . . . . . . .

jiri

2006-06-22 05:42

administrator   ~0007369

Last edited: 2006-06-22 05:42

3) I'd prefer to make it similar to synchronization functionality (as it currently is). Not only it's good because we can make it both look the same way, but also I like the fact that you can configure supported formats (like: mp3, ogg, wma) and then only set rules for incompatible formats (e.g. incompatible between 100..200, incompatible >200, etc.).

rusty

2006-06-30 21:50

administrator   ~0007395

OK, let's hold off on 3) for now. I'll review the issue again when everything else is fixed.

Note: my problem isn't so much the presentation of this information, but rather the fact that the user has to go to 2 different locations to configure a rule, when we could easily do it via a single interface.

Ludek

2006-07-18 23:53

developer   ~0007541

Last edited: 2006-07-19 00:26

I must disagree with couple of things:

1.
'There will be probably not enough free space to convert all tracks. Check the temporary directory free space'
[Cancel] [Retry]
is more useful than
'There is insufficient space in the temporary directory to convert all tracks. Do you want to continue?' [[Yes]] [Cancel]
cause a user can check the space in temp dir and free this space (by deleting some files on the disk where temp dir. appears)
and then click [Retry] button to be able to continue with sufficient space. He cannot continue without sufficient space for pre-converted files!


2. Re: The Auto-Convert option makes more sense on the 'Choose Disc Format' panel, just below the 'Burn .m3u Playlists' option.

I've investigated major portion of time to improve total size counting on 'Edit data CD/DVD content' page so that a user could comfortable calculate
the total file size while he is changing his AC settings (by using [options] button, enabling AC, checking items).
( By improving I mean mainly speeding up size counting by keeping checked items in memory and by using of hashing structures.)
--> i.e. I think that its place is very useful and right

In addition it does not take any necessary room like in case of the next page.



Re:
'There was a problem during Auto-Conversion'
[Ok]
This message occurs only once, after some of converted files has failed or after a user has clicked
[Cancel] button in case of message
'There won't be probably enough free space to convert all tracks. Check the temporary directory free space'
[Cancel] [Retry]


Re: 'Select formats supported by the the device that will play this CD/DVD.'
I would like the text to include word 'compatible' or 'compatible formats' cause in AC settings a user sets Format on PC: 'incompatible'.

Ludek

2006-07-19 14:46

developer   ~0007544

And another thing is that text used for MM Gold (in case of Synchronization) is
'The standard version of MediaMonkey will synchronize tracks exactly as they exist in your Library. To synchronize and convert unsupported or high-bandwidth tracks on-the-fly to formats that are supported and take up less memory on your device, please upgrade to MediaMonkey Gold.'

Please, give me another text which could be used in case of burning, thx.

rusty

2006-07-19 17:03

administrator   ~0007546

Here's a new string that can be used for burning and synching--i.e. just replace the current string with this:

The standard version of MediaMonkey burns and synchs tracks in the format in which they exist in your Library. To convert tracks on-the fly to different formats or to lower bitrates, please upgrade to MediaMonkey Gold.

Ludek

2006-07-20 00:10

developer   ~0007552

Ok, the text is updated. Please, review my 7541 note.

rusty

2006-07-20 05:32

administrator   ~0007555

Last edited: 2006-07-20 05:34

1) Makes sense, my original intent was mainly for gramatical clarify, but this revised suggestion should meet all needs:

'There isn't enough free space to convert all tracks. Please clear space in the temporary directory. [[Retry]] [Cancel]'

2) I still think that auto-convert makes more sense on the following panel. On the first panel user selects _what_ music. On the next panel, they select _how_ the music is to be transfered.

Let's leave it for now, though--I'll re-evaluate in the next build.

3) Re. 3) I'm ok with holding off on any major changes. Main thing to keep in mind is that the UI should be similar to 0002107. Also, keep in mind 0002108--it would be good if the code could be reused.

Ludek

2006-07-25 19:29

developer   ~0007603

All have been resolved except 0002108. This will be solved in that bug.