Author Topic: xaimus' ramping patch  (Read 5858 times)

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
xaimus' ramping patch
« 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline LPChip

  • Administrator
  • *****
  • Posts: 5,102
    • http://lpchip.nl
  • Operating System: Windows 10 Pro x64
Re: xaimus' ramping patch
« Reply #1 on: July 17, 2011, 13:54:24 »
Can you be a bit more specific?
"Heh, maybe I should've joined the compo only because it would've meant I wouldn't have had to worry about a damn EQ or compressor for a change. " - Atlantis
"yes.. I think in this case it was wishful thinking: MPT is makng my life hard so it must be wrong" - Rewbs

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #2 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline LPChip

  • Administrator
  • *****
  • Posts: 5,102
    • http://lpchip.nl
  • Operating System: Windows 10 Pro x64
Re: xaimus' ramping patch
« Reply #3 on: July 17, 2011, 18:04:24 »
For what VSTi is this patch?
"Heh, maybe I should've joined the compo only because it would've meant I wouldn't have had to worry about a damn EQ or compressor for a change. " - Atlantis
"yes.. I think in this case it was wishful thinking: MPT is makng my life hard so it must be wrong" - Rewbs

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #4 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline LPChip

  • Administrator
  • *****
  • Posts: 5,102
    • http://lpchip.nl
  • Operating System: Windows 10 Pro x64
Re: xaimus' ramping patch
« Reply #5 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?
"Heh, maybe I should've joined the compo only because it would've meant I wouldn't have had to worry about a damn EQ or compressor for a change. " - Atlantis
"yes.. I think in this case it was wishful thinking: MPT is makng my life hard so it must be wrong" - Rewbs

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #6 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline LPChip

  • Administrator
  • *****
  • Posts: 5,102
    • http://lpchip.nl
  • Operating System: Windows 10 Pro x64
Re: xaimus' ramping patch
« Reply #7 on: July 18, 2011, 09:51:20 »
Aaaaaah. Now you're telling me!

Thanks. The topic makes much much more sense now. :nuts:
"Heh, maybe I should've joined the compo only because it would've meant I wouldn't have had to worry about a damn EQ or compressor for a change. " - Atlantis
"yes.. I think in this case it was wishful thinking: MPT is makng my life hard so it must be wrong" - Rewbs

Offline jmkz

  • Workaholic artist
  • ***
  • Posts: 212
  • Gender: Male
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #8 on: July 21, 2011, 23:57:54 »
How to apply this patch? I tried some ways but unsuccessfully.

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #9 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline jmkz

  • Workaholic artist
  • ***
  • Posts: 212
  • Gender: Male
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #10 on: July 23, 2011, 01:30:48 »
LOL, I tried to do that, but nope, it doesn't worked.

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #11 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Offline jmkz

  • Workaholic artist
  • ***
  • Posts: 212
  • Gender: Male
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #12 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?

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 7,104
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
Re: xaimus' ramping patch
« Reply #13 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.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.