Bug 2014 – Regular Interval Labels fails

••• Introduction •••
••• Keywords •••

Audacity 3.0.3 development began 19th April 2021

Audacity Bugzilla

Bug 2014 - Regular Interval Labels fails


Summary:	Regular Interval Labels fails

Status:	RESOLVED FIXED

Product:	Audacity
Classification:	Unclassified
Component:	Nyquist
Version:	2.3.0
Hardware:	Per OS All

Importance:	P3 RepeatableAll
Assigned To:	Steve Daulton

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2018-10-22 08:07 UTC by Steve Daulton
Modified:	2018-11-10 05:44 UTC (History)
CC List:	4 users (show)

See Also:
Steps To Reproduce:	Try to apply "Regular Interval Labels"
Release Note:	Expected to be closed before release.
First Git SHA:
Group:	---
Workaround:
Closed:

Flags:	stevethefiddle: Regression+ petersampsonaudacity: Test‑OK‑Win+ petersampsonaudacity: Test‑OK‑Mac+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Steve Daulton 2018-10-22 08:07:43 UTC

"Regular Interval Labels" fails with:
Nyquist returned the value:'N'
Where "N" is the number of digits specified.

The problem is due to the plug-in being changed from a "Analyze" effect to a "Tools" type.

There are also other problems with this plug-in, including redundant translation strings.

I'll take this bug and look at the other issues at the same time.

Comment 1 Steve Daulton 2018-10-22 18:55:38 UTC

Fixed at:
https://github.com/audacity/audacity/commit/72fbf1b

A possible further enhancement would be to rewrite this effect using AUD-DO commands so that the requirement for an audio selection can be removed, but that is an enhancement beyond the scope of this bug.

Comment 2 Peter Sampson 2018-11-06 10:18:57 UTC

Testing on W10 with audacity-2.3.1-alpha-207-4c76e598d5859dc172e063d37287cc510fa7850f

This now seems to wok correctly as expected on W10

And testing on 2.3.0 I confirm the behavior Steve reports in Comment #0

Comment 3 Peter Sampson 2018-11-10 05:44:25 UTC

Testing on macOS 10.14.1 Mojave with Steve's 2.3.1 alpha build of 09Nov18

This now seems to work correctly as expected on MC