Can't repeat 1 times on Audacity 2.1.1

Hi guys,
Just update to Audacity 2.1.1 today. It throws an error “Validation error: Not in range” when I try to repeat track 1 times (effect. It only allows to repeat greater than 2 times. Can I change it somewhere or it’s a bug? Please help, thank you in advance.
audacity error.PNG

It’s a bug.
I thought that this had been fixed several weeks ago, but apparently not.
The workaround is to copy and paste for a single repeat rather than use the Repeat effect. Ctrl+C to copy, Ctrl+V to paste.

The quickest way to copy and paste in your example is:

  1. Ctrl+C
  2. Set “Length” in the Selection toolbar (at bottom of main window) to 00h 00m 00.000s
  3. Ctrl+V

This does raise the question, (and since this needs to be fixed I think it’s a good time to address this question), should “Number of times to repeat” mean:
a) the number of additional repeats
b) the total number of repeats.

In other words, if you select some audio and set the “Number of times to repeat” to 16, should you end up with 16 x the original or 17 x the original? Previous versions of Audacity have given 17 x the original.

I think it should be the former “the number of additional repeats” - as basically you are taking the selected audio and making repeats - i.e. original plus repeats. I suspect that the intent may be clearer if the default was set to 1 rather than 10 as it is now.

If it were to be option b) then that text in the dialog should really be changed to something like “Number of instances” rather then the existing “Number of times to repeat” to give more clarity of the intent.

An by the way its always been option a) the number of additional repeats in Audacity - I just tested my vintage 1.2.6 version :ugeek:

But yes there are bugs:

  1. in 2.1.0 you can specify zero repeats , this is not trapped with an out of range error message - rather it creates one repeat - this is clearly wrong

  2. in 2.1.1 RC2 if you specify either zero or one you get the Validation error: Not in range message. This is correct behavior if the user should ever type zero in there, but wrong when the user specifies 1 repeat.

It is not worth logging the former - but have you already logged the 2.1.1 RC2 bug on Bugzilla Steve? I suspect that this will remain unfixed (and thus release noted) for 2.1.1 now - but for my money it is an irritating bug (even thought there is a simple copy&paste workaround). And note that this is a regression on 2.1.0 where it is possible to make a single repeat.

Peter.

P.S. I can’t update the Manual right now as it is in freeze mode while we are in the release process for 2.1.1 until we make the release - but I will try to add some further clarity to the Manual page at least. And maybe, Steve, you could look at clarifying the text in the dialog then too? I had thought that the dialog and the Manual expressed the intent with sufficient clarity - but maybe not given this Forum thread.

OK I see, Steve, that you have already posted this bug and Gale has already replied: http://bugzilla.audacityteam.org/show_bug.cgi?id=1060

I’ve added my comments too.

Peter.

+1.
Currently it says (as the default):

Number of times to repeat: [ 10 ]

but then when I play it back, it plays 11 times :confused: - that is not obvious. I bet that has caught out most users when they use the effect for the first time.

If it just said (as the default):

Number of repeats: [ 1 ]

which then produces “1 repeat” (+ the original), then I think that would be clear and obvious for first time users.

Also,

  1. I think it would be worth showing the current selection duration as well as the new duration.
  2. The duration time could be in the same format as selected in the Selection toolbar.

Thus the default would be something like:

Number of repeats: [ 1 ]
Current selection duration: 00h 00m 30.000s
New selection length: 00h 01m 00.000s

Yes, it would be much easier to see and count one original plus one repeat when you first try it out - 10 versus 11, not so obvious

+1 to that too

Peter

If users are getting confused as has been suggested, then I think only changing to “Number of repeats” and no other changes would if anything be less clear than now.

So I think the text should be “Number of repeats to add” or similar and we don’t add the current selection length, OR if we add the current selection length then “Number of repeats” is OK.

I also thought about adding the current selection length, but it might make the effect “look” more complex than necessary.

I agree that a default of “1” repeat makes what will happen clearer.

Gale

Looks good to me:
fullwindow-Repeat-001.png

I do like the idea of using the current Selection Toolbar format for the displayed length. If we did that the default format would give us text that looked like:


Number of repeats: [ 1 ]

Current selection length: 00 h 00 m 03.367 s
New selection length: 00 h 00 m 06.734 s

I can only vote one way or the other, so I would vote not to add current selection length on the grounds it adds a lot of extra text versus adding “to add”.

But I can be outvoted on this one.


Gale

We could cut down the amount of text by putting “Current length:” and “New length:” rather than “Current selection length:” …
However, I don’t think that it looks excessive to include the word “selection”, and it is perhaps a bit clearer.

I’m in favour of including the current length, not only because it clarifies the issue of “new repeats” vs “total repeats”, but because it’s useful to know the selection length when using this effect, and the Selection toolbar may be set to show End time rather than Duration. I like that the interface shows all of the relevant information at a glance.

+1 for that (I like good labelling on the tin, as you know)

and I quite like the idea of seeing the original selection length and the new after-repeat length - and I like the idea of using thus users current format setting from Selection Toolbar.

Peter.

I certainly don’t object to including the current length, and it is analogous with “From” and “To” in the Change Speed/Tempo/Pitch effects.

I just don’t think for this effect it’s absolutely necessary versus the extra text displayed. But up to you, since no doubt you’ll be fixing this.


Gale

I think that “to add” is unnecessary verbiage, especially if we show the original length as well as the new length, but I don’t feel strongly about it, so here it is (out of the box settings):
fullwindow-Repeat-003.png
Question: Should the effect remember the last used setting from one session to the next, or should it default to 1 repeat when Audacity is restarted?

Mock-up looks good to me

Don’t most other effects remember the last used setting when re=opening Audacity (provided the pluginxxx.cfg files haven’t been deleted)?

But in this case I think I would be minded to favour reversion to the default setting of 1 repeat.

Peter.

Yes.

We can have it either way - we just need to decide.
I’m mildly in favour of remembering the last setting, mostly on the grounds of consistency.
(In fact, since Leland’s update, the effect already remembers the last used setting, but the remembered setting is not yet used).

My suggestion if you recall was to add “to add” OR add the current selection length.

Gale

I think it should remember the last setting in the next Audacity session. This should make more sense for those creating loops, as well as being consistent with most effects.


Gale

Cool, then I’ll go for adding the current selection length.

+1 for that too.

Both of these enhancements will be post 2.1.1 release.

@dejavu494
Thanks for you observation. All being well, when released, Audacity 2.1.1 will be able to do 1 repeat.