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

usability #846

Open
4 tasks done
p0nce opened this issue Apr 14, 2024 · 6 comments
Open
4 tasks done

usability #846

p0nce opened this issue Apr 14, 2024 · 6 comments
Labels
Enhancement This issue is about a new feature rather than a bug.

Comments

@p0nce
Copy link
Collaborator

p0nce commented Apr 14, 2024

  • readParamdouble doesn't exist, despite parameters being double internally
  • I think some setNormalize option doesn't clamp, I don't remember which one exactly
    Specifically: setFromGUINormalized is a public API that exists for IntegerParameter and FloatParameter
    Calling them with out of [0 .. 1] should clamp them, but also create a debug time assertion, instead of just having the debug time assertion. It is because the primary user for this call is sliders and knobs of all sort, who maintain complicated dragging invariants. Now outside 0 to 1 is allowed.
  • A Bessel 2-pole in iir.d, since it's very useful
  • UISlider has sensitivity as a property, instead of a Wren-accessible script property (variable), which isn't same as UIKnob. Change that to regular variable.
@p0nce p0nce added the Enhancement This issue is about a new feature rather than a bug. label Apr 14, 2024
@p0nce p0nce closed this as completed Apr 21, 2024
@p0nce
Copy link
Collaborator Author

p0nce commented Apr 22, 2024

  • convertDecibelToLinearGain called with int is an issue. Specialize the function instead.

@p0nce p0nce reopened this Apr 22, 2024
p0nce pushed a commit that referenced this issue May 5, 2024
…eird D property. This is more in line with UIKnob. See #846
@p0nce p0nce closed this as completed May 24, 2024
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 11, 2024

  • It should be possible to feedBuffer a delay line with a buffer larger than the delay line. This is a common use case in case of silence detection. In this case, only feed the last size-of-delayline samples.

@p0nce p0nce reopened this Jun 11, 2024
@p0nce p0nce closed this as completed Jun 11, 2024
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 17, 2024

  • VERY easy and mysterious trap with EnumParameter for the longest time, one hour lost

This doesn't crash at runtime:

static immutable string[] names = ["V1", "V2", "V3"];
params.pushBack( mallocNew!EnumParameter(paramSomething, "MyEnum", names, 1));

But this crash:

static immutable string[3] names = ["V1", "V2", "V3"];
params.pushBack( mallocNew!EnumParameter(paramSomething, "MyEnum", names, 1));

find a way to avoid that

@p0nce p0nce reopened this Jun 17, 2024
@p0nce
Copy link
Collaborator Author

p0nce commented Jun 24, 2024

  • Because of lack of separation of UI and DSP, .visibility can be called by audio thread (like setDirty). Ensure it is possible and won't crash.

@p0nce
Copy link
Collaborator Author

p0nce commented Jun 29, 2024

  • Need a way to not clear UIBufferedElementRaw and UIBufferedElementPBR, since the UIElement owns that buffer, it could well choose to not have this clearing (example: to use text-mode efficiently) and reuse the older buffer content! flagOpaque?

@p0nce
Copy link
Collaborator Author

p0nce commented Jul 10, 2024

  • Wren doesn't support vec2f, vec3f, vec4f, vec2d, vec3d, vec4d as @ScriptProperty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement This issue is about a new feature rather than a bug.
Projects
None yet
Development

No branches or pull requests

1 participant