Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calf plugins does not work well with Carla. #260

Open
MaciejMalczyk opened this issue Jul 18, 2020 · 29 comments
Open

Calf plugins does not work well with Carla. #260

MaciejMalczyk opened this issue Jul 18, 2020 · 29 comments
Milestone

Comments

@MaciejMalczyk
Copy link

I discovered that some parameters are not well controllable in Carla plugin host via Carla's parameters tab. Problem is in very non linear (mostly logarythmic) way of controlling plugin parameters even if knob has linear behavior in plugin GUI.

I think this short video will clarify everything about problem:
https://drive.google.com/file/d/1Mfj0_AU1eFmwse8R1ZmIgKFpxYCRunB7/view?usp=sharing

@falkTX
Copy link
Collaborator

falkTX commented Jul 18, 2020

This is likely an issue with Carla and not Calf.

@MaciejMalczyk
Copy link
Author

MaciejMalczyk commented Jul 18, 2020

Maybe but the same happens in Ardour when i try to change value with "Ardour slider". Does Carla/Ardour read raw values from plugins?
I have one idea to overcome this - custom progression of sliders values. In LMMS for example you can choose knob behavior between logarithmic and linear. LMMS way in this case is of course not a grate solution because of low customization of values progression.

Anyway, I think it will be better to move Carla specific discussion to Carla's github.

@falkTX
Copy link
Collaborator

falkTX commented Jul 18, 2020

If Ardour also has the issue, it is likely that then it is on Calf side after all.
LV2 plugins should expose a logarithmic property on the ports that behave that way, so the host knows to give it a log scale.
I do not recall if Carla generic editor supports this, but Ardour is very likely to.

@JohannesLorenz
Copy link
Collaborator

This seems to only affect gain ports, right? Those should be fixed in PR #342 . Would be nice if you could test.

@JohannesLorenz
Copy link
Collaborator

@wujekbrezniew Friendly reminder: Can you please check if your problem has been fixed in #342 ?

@MaciejMalczyk
Copy link
Author

Hi,
Sorry for not responding for so long.
I tested this PR (branch #320) and nothing changed.
Video

@JohannesLorenz

@JohannesLorenz
Copy link
Collaborator

@wujekbrezniew Thanks for replying. Which parameter (in which plugin) behaves wrong with the #320 branch?

@MaciejMalczyk
Copy link
Author

MaciejMalczyk commented Feb 17, 2024

@JohannesLorenz I tested Calf Compressor. Attack, release, input, makeup gain. I put a link to video i today's response that shows a problem.

@JohannesLorenz
Copy link
Collaborator

JohannesLorenz commented Feb 17, 2024

Thanks again, @wujekbrezniew . Sorry, I oversaw the video link. What I see is that in the video, all these sliders move linear in the GUI, however, all have

@prefix epp: <http://lv2plug.in/ns/ext/port-props#>
lv2:portProperty epp:logarithmic ;

This is strange. I can only see 3 possible reasons for this:

  • Did you forget one of the pull/configure/make/make install steps?
  • Did you accidentally still use an old version of calf? Maybe there are concurrent CALF versions? Can you first make sure there is aboslutely no CALF compressor visible in Carla, and only then set the LV2_PATH to your PR's install directory and restart Carla?
  • Your version of Carla does not detect lv2:portProperty epp:logarithmic. That sounds really strange though. Can you try it with at least one other daw?

@MaciejMalczyk
Copy link
Author

MaciejMalczyk commented Feb 17, 2024

@JohannesLorenz
My Carla is built from this commit:
falkTX/Carla@68bbca2
I will try to build latest one and find out if there is a problem.

I build calf by doing ./configure.sh, make -16, sudo make install. I removed calf-plugins packages from apt so i'm sure that i use compiled ones in Carla. I checked if your changes are in my local repo and they are.
My lv2-dev package is 1.18.10 from kxstudio so i changed lv2 headers to: #include "lv2/core/lv2.h" from #include "lv2/lv2plug.in/ns/lv2core/lv2.h" and so on in other files to compile it against newest version on lib.

I checked also Ardour that i built from commit:
Ardour/ardour@ad51c7c
And it works properly. Video below.
link

@MaciejMalczyk
Copy link
Author

Tested Carla built from latest commit. Still the same problem.

@JohannesLorenz
Copy link
Collaborator

@wujekbrezniew Thanks! IIRC the problem occurs with Carla, but not with Ardour. That sounds like a bug in Carla, still strange, because Carla is very old software.

Can you please test if the CALF from our master works with Ardour? If the mentioned issue occurs there, too, then #342 fixed an issue here.

@MaciejMalczyk
Copy link
Author

MaciejMalczyk commented Feb 17, 2024

@JohannesLorenz
Can confirm that in Ardour Calf compressor from master has an issue and from #320 branch works well.
I was checking also Ratio knob - for me it is still hard to control but i dont know exactly what is responsible for that. Video below how it works:
video

@JohannesLorenz
Copy link
Collaborator

@wujekbrezniew Thanks for the info. Considering that, I think we should file a bug report (including the ttl file of the compressor) to Carla. It might still be a bug on our side that Ardour just manages well, but in all cases, it should be reported. Do you want to do that, or should I?

About the Ratio knob: From the video, it looks like the UI displays this knob logarithmic, while the slider on the left is linear? Which means we should mark this slider as "logarithmic", too?

@MaciejMalczyk
Copy link
Author

@JohannesLorenz I think i will be better if you file bug report to Carla, because you have more technical knowledge.
You can mark the knob as logarythmic and then i will try to test that. Maybe i will try to test other plugins to find out if there are some other problems like this.

@JohannesLorenz
Copy link
Collaborator

JohannesLorenz commented Feb 18, 2024

@wujekbrezniew Pushed. This knob is a bit strange though, because it says that the slider should have a "fake infinity". Can you please check if the knob matches the GUI well, also for the upper values?

For submitting a report in Carla, I would need the lib/lv2/calf.lv2/Compressor.ttl from your install directory.

@MaciejMalczyk
Copy link
Author

This is from build of #320 branch. Extension changed from ttl to txt.
Compressor.txt

@JohannesLorenz
Copy link
Collaborator

This is from build of #320 branch. Extension changed from ttl to txt. Compressor.txt

Reported, thanks!

Can you please check if the knob matches the GUI well, also for the upper values?

Just as a friendly reminder, this is still open.

@MaciejMalczyk
Copy link
Author

Can you please check if the knob matches the GUI well, also for the upper values?

You can check it on video. I'm not 100% shure if this is working well or not c:
https://drive.google.com/file/d/1pMqyJF1RH1_BF-F2yEuvI60omquftms7/view?usp=sharing

If i remember exactly there are other plugins that have simmilar problem to compressor. I will try to find out those one's and theirs problematic params and post here (or maybe open new issue?).

@JohannesLorenz
Copy link
Collaborator

If i remember exactly there are other plugins that have simmilar problem to compressor. I will try to find out those one's and theirs problematic params and post here (or maybe open new issue?).

Hi @wujekbrezniew , do you still plan to do this (friendly reminder)?

@JohannesLorenz
Copy link
Collaborator

I'm not 100% shure if this is working well or not c:

I was also not sure but to me it looks OK now. It seems to slide steadily over a logarithmic scale.

@MaciejMalczyk
Copy link
Author

Hi. I thought that this was closed c:

I will try to test other plugins in weekend and post here problems that i found.

@MaciejMalczyk
Copy link
Author

MaciejMalczyk commented May 19, 2024

I tested 4 plugins: Tape Simulator and 5, 8, and 30 Equalizers. Both in Ardour and in Carla (built from git). In all cases if plugin has some dial connected to gain or level, controlling it from host slider is impossible - linear movement make logarithmic change in value. Basically if something is in dB it has this bug.

For some reason in Carla frequency dial for filters also works logarithmic. Ardour does not have this problem.

@JohannesLorenz
Copy link
Collaborator

controlling it from host slider is impossible - linear movement make logarithmic change in value

Can you please make a video, so I understand better what the problem is? Thanks on advance.

@MaciejMalczyk
Copy link
Author

CarlaCalfEq5LevelBug.mp4
ArdourCalfEq5LevelBug.mp4

@JohannesLorenz
Copy link
Collaborator

Is this only a visualization problem of the GUI, or does this even sound as wrong as it looks?

@sven-frotscher
Copy link

Carla does what you would expect from its knob/slider sweeping a linear range (i.e. unity gain is very far left) and probably disregards lv2:portProperty epp:logarithmic ; altogether. I'd say the Calf GUI is working just fine and Carla is missing the "logarithmic control" feature.

@JohannesLorenz
Copy link
Collaborator

I'd say the Calf GUI is working just fine and Carla is missing the "logarithmic control" feature.

Carla indeed has problems with logarithmic control, but that does not explain the issues with Ardour.

Is this only a visualization problem of the GUI, or does this even sound as wrong as it looks?

@MaciejMalczyk Friendly reminder that I could still need your input here, thanks!

@JohannesLorenz
Copy link
Collaborator

As this still requires response from the OP and the fix might be non-trivial, I move this to a new 0.90.5 milestone

@JohannesLorenz JohannesLorenz modified the milestones: 0.90.4, 0.90.5 Aug 3, 2024
@JohannesLorenz JohannesLorenz pinned this issue Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants