ModPlug Central

OpenMPT => Development Corner => Topic started by: Saga Musix on July 17, 2011, 12:00:54

Title: xaimus' ramping patch
Post by: Saga Musix on 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.
Title: Re: xaimus' ramping patch
Post by: LPChip on July 17, 2011, 13:54:24
Can you be a bit more specific?
Title: Re: xaimus' ramping patch
Post by: Saga Musix on 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.
Title: Re: xaimus' ramping patch
Post by: LPChip on July 17, 2011, 18:04:24
For what VSTi is this patch?
Title: Re: xaimus' ramping patch
Post by: Saga Musix on 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.
Title: Re: xaimus' ramping patch
Post by: LPChip on 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?
Title: Re: xaimus' ramping patch
Post by: Saga Musix on July 17, 2011, 23:00:27
It is a patch, not a patch. :nuts: 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.
Title: Re: xaimus' ramping patch
Post by: LPChip on July 18, 2011, 09:51:20
Aaaaaah. Now you're telling me!

Thanks. The topic makes much much more sense now. :nuts:
Title: Re: xaimus' ramping patch
Post by: jmkz on July 21, 2011, 23:57:54
How to apply this patch? I tried some ways but unsuccessfully.
Title: Re: xaimus' ramping patch
Post by: Saga Musix on 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.
Title: Re: xaimus' ramping patch
Post by: jmkz on July 23, 2011, 01:30:48
LOL, I tried to do that, but nope, it doesn't worked.
Title: Re: xaimus' ramping patch
Post by: Saga Musix on 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.
Title: Re: xaimus' ramping patch
Post by: jmkz on 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?
Title: Re: xaimus' ramping patch
Post by: Saga Musix on 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.