xaimus' ramping patch

Saga Musix · July 17, 2011, 12:00:54

This patch allows to use separate values for ramping samples in and out, for even higher fidelity and punchier transients. Still need to look into everything but it seems to work well so far.

LPChip · July 17, 2011, 13:54:24

Can you be a bit more specific?

Saga Musix · July 17, 2011, 16:57:04

It allows for separate ramp-in / ramp-out times for samples. You will hear more about that when it actually made its way into the main code branch, but for now this information should be enough. If people want to experiment with the patch, they can do so.

LPChip · July 17, 2011, 18:04:24

For what VSTi is this patch?

Saga Musix · July 17, 2011, 20:36:20

Was I not being clear enough that this patch is for samples only? Ramping is not applied to VSTis, they have their own audio processing.

LPChip · July 17, 2011, 22:04:47

Huh? You have patches other than VSTi patches? are the samples for like a soundfont player? and if so, which one?

Saga Musix · July 17, 2011, 23:00:27

It is a patch, not a patch.

I.e. a software patch. Something that fixes software. In this case, it fixes OpenMPT's code. Hence I put this in the OpenMPT development corner.

LPChip · July 18, 2011, 09:51:20

Aaaaaah. Now you're telling me!

Thanks. The topic makes much much more sense now.

jmkz · July 21, 2011, 23:57:54

How to apply this patch? I tried some ways but unsuccessfully.

Saga Musix · July 22, 2011, 17:58:19

If you use TortoiseSVN, right-click the source folder, choose "Apply patch" and pray that there are no conflicts.

jmkz · July 23, 2011, 01:30:48

LOL, I tried to do that, but nope, it doesn't worked.

Saga Musix · July 23, 2011, 11:20:30

Then you will have to resolve conflicting lines. Or just wait until it is used in the main branch at some point.

jmkz · August 24, 2011, 18:11:53

I have fixed some lines in patch, now is detected by TortoiseSVN. Needs get some code lines updated by the latest revision.

Which revision were used for this patch?

Saga Musix · August 24, 2011, 18:20:07

I wouldn't recommend using the patch as-is, I have already done most of the necessary modifications to implement the feature in a proper way but I won't commit those changes too soon.