View Issue Details

IDProjectCategoryView StatusLast Update
0003461ardourbugspublic2020-04-19 20:14
Reportermanmachine Assigned Tocth103  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version2.8.11 
Target Version2.8.12 
Summary0003461: Deafening sound output at region cuts
DescriptionAfter regions are cut and joined in certain ways,the playback at joint is ear deafening output(600db!).Link of a video demonstrating the operation and the problem is here.

http://www.keepandshare.com/doc/2251321/playback-error-ogv-september-19-2010-6-53-pm-7-2-meg?da=

Sorry, the file is more than 5000k size

This happens only in ArdourVST 2.8.11 not in Ardour 2.8.11

Forum thread : http://ardour.org/node/3717
Additional InformationThis occurs even there are no plug-ins used. Occurs on imported audio regions, and I don't believe on recorded regions.Important: The problem is always reproducible if the snap is set GRID

Workaround:
Provide hair-gap between the regions, where problem occurs.(Shorten the preceding region very minutely)
TagsNo tags attached.

Activities

manmachine

2010-09-19 14:45

reporter   ~0009098

Sorry for my ignorance.The problem is there on all audio materials and not as reported earlier(ie. I reported it as it is there on imported audio only)

cth103

2010-09-19 16:09

administrator   ~0009100

Can you attach the file ~/.ardour2/ardour.rc to this report?

2010-09-19 18:12

 

ardour.rc (3,198 bytes)

manmachine

2010-09-19 18:13

reporter   ~0009102

File attached

2010-09-19 19:11

 

3461.patch (959 bytes)   
diff --git a/libs/ardour/curve.cc b/libs/ardour/curve.cc
index f2ab337..94b6ed8 100644
--- a/libs/ardour/curve.cc
+++ b/libs/ardour/curve.cc
@@ -312,7 +312,7 @@ Curve::_get_vector (double x0, double x1, float *vec, int64_t veclen)
 		return;
 	}
 
- 	if (npoints == 1 ) {
+ 	if (npoints == 1) {
  	
  		for (i = 0; i < veclen; ++i) {
  			vec[i] = events.front()->value;
@@ -355,17 +355,18 @@ Curve::_get_vector (double x0, double x1, float *vec, int64_t veclen)
 
 	rx = lx;
 
+	/* note: if there are veclen elements in the output,
+	   there are only veclen-1 steps between them.
+	*/
+	
 	if (veclen > 1) {
-
-		/* note: if there are veclen elements in the output,
-		   there are only veclen-1 steps between them.
-		*/
-
 		dx = (hx - lx) / (veclen-1);
-
-		for (i = 0; i < veclen; ++i, rx += dx) {
-			vec[i] = multipoint_eval (rx);
-		}
+	} else {
+		dx = 0;
+	}
+	
+	for (i = 0; i < veclen; ++i, rx += dx) {
+		vec[i] = multipoint_eval (rx);
 	}
 }
 
3461.patch (959 bytes)   

cth103

2010-09-19 19:11

administrator   ~0009103

I believe the attached patch fixes this. Could you give it a try?

manmachine

2010-09-19 20:06

reporter   ~0009104

I'm sorry,I don't know, how/what to patch. I'm a musician.Could you please help me out? could a patchwork like this be performed by a non-technical person?

cth103

2010-09-19 20:18

administrator   ~0009105

How did you get Ardour? Did you download the binary from ardour.org?

manmachine

2010-09-19 20:39

reporter   ~0009106

Last edited: 2010-09-19 20:54

Yes, I did download ,build ardour from source

cth103

2010-09-19 20:52

administrator   ~0009107

You'd have to build Ardour from source to apply this patch, which isn't particularly easy if you haven't done it before. I'll see if we can maybe arrange a release with this fix if others can verify that it helps.

manmachine

2010-09-19 21:00

reporter   ~0009108

I have no problem in building Ardour , as I did it many times successfully,in fact from version 2.8.4.If you consider this is not sufficient to do patching, then I wait for a release with fix.Thank you for responding

cth103

2010-09-19 21:13

administrator   ~0009109

Ah, if you have built ardour before then it's easy. Download the patch from this bug (3461.patch). At a terminal, move into the top level of the ardour source directory, and then type

patch -p1 < /path/to/3461.patch

where /path/to/3461.patch is where you downloaded the patch to. Then rebuild as normal.

manmachine

2010-09-19 21:21

reporter   ~0009110

I'll give a try.Will be back soon.Thank you very much.

manmachine

2010-09-19 22:43

reporter   ~0009112

Great! That worked for me.Thank you so much.

cth103

2010-09-20 00:55

administrator   ~0009113

Cool, thanks for checking and for the excellent bug report.

cth103

2010-09-20 00:58

administrator   ~0009115

Patch applied to 3.0 SVN.

paul

2010-09-20 13:37

administrator   ~0009116

applied to 2.X SVN

tartina

2012-03-19 11:15

reporter   ~0012977

I still have this issue with 2.8.12, from Fedora 16 repository. I think it's more likely when crossfades are very small. I usually paste drum samples in ardour tracks and when the length of the samples are such that next sample start just when the previous ends, the noise is likely to happen. I have to slightly shorten the preceding sample to solve this problem.

Guido

system

2020-04-19 20:14

developer   ~0022228

Issue has been closed automatically, by Trigger Close Plugin.
Feel free to re-open with additional information if you think the issue is not resolved.

Issue History

Date Modified Username Field Change
2010-09-19 14:06 manmachine New Issue
2010-09-19 14:45 manmachine Note Added: 0009098
2010-09-19 16:09 cth103 Note Added: 0009100
2010-09-19 16:09 cth103 Status new => feedback
2010-09-19 18:12 manmachine File Added: ardour.rc
2010-09-19 18:13 manmachine Note Added: 0009102
2010-09-19 19:11 cth103 File Added: 3461.patch
2010-09-19 19:11 cth103 Note Added: 0009103
2010-09-19 20:06 manmachine Note Added: 0009104
2010-09-19 20:18 cth103 Note Added: 0009105
2010-09-19 20:39 manmachine Note Added: 0009106
2010-09-19 20:52 cth103 Note Added: 0009107
2010-09-19 20:54 manmachine Note Edited: 0009106
2010-09-19 21:00 manmachine Note Added: 0009108
2010-09-19 21:13 cth103 Note Added: 0009109
2010-09-19 21:21 manmachine Note Added: 0009110
2010-09-19 22:43 manmachine Note Added: 0009112
2010-09-20 00:55 cth103 cost => 0.00
2010-09-20 00:55 cth103 Note Added: 0009113
2010-09-20 00:55 cth103 Target Version => 2.8.12
2010-09-20 00:58 cth103 Note Added: 0009115
2010-09-20 00:58 cth103 Status feedback => confirmed
2010-09-20 13:37 paul Note Added: 0009116
2010-09-20 21:19 cth103 Status confirmed => resolved
2010-09-20 21:19 cth103 Resolution open => fixed
2010-09-20 21:19 cth103 Assigned To => cth103
2012-03-19 11:15 tartina Note Added: 0012977
2020-04-19 20:14 system Note Added: 0022228
2020-04-19 20:14 system Status resolved => closed