New LAF Properties for Modulation Display: Selected Source vs Accumulated Range

[Christoph Hart]

@DanH This is exactly the workflow & scope of fix that should be covered by the new contribution workflow, so I think it would be best to get you up and running there ASAP.

For now just make a manual pull request with the changes, I'll review it and merge if OK.

[DanH]

@Christoph-Hart Done. It also includes a crash fix I was experiencing regarding IntensityTextConverter

[Christoph Hart]

@DanH Just looked at it and I'm a bit disappointed - the code quality is far worse than what I'm able to oneshot without any guidance with my tuned workflow. I'm afraid we have to postpone this until the procedure is ready.

If it fixes your immediate problem, then just take the fix and use it locally, but I would have to fully rewrite every single line of code to match the coding style / structure and quality of the existing code.

[DanH]

@Christoph-Hart bad Claude! I'll pass on the feedback!! Obviously it doesn't have access to the whole codebase but it should be able to match the code quality of the files it read.... Will stick with my fork for now.

[Christoph Hart]

@DanH said in New LAF Properties for Modulation Display: Selected Source vs Accumulated Range:

it doesn't have access to the whole codebase but it

Lol why? that's the first thing you need to do, start the claude session from the HISE repo folder, it will then just explore and pull in whatever it takes. A full codebase exploration including the fix of this scope should stay below 60k tokens.

[DanH]

@Christoph-Hart on web for now....

[Christoph Hart]

@DanH ah ok, well that's precisely the difference between "lol AI cannot do s***" and "boy we're f***ed"....

[DanH]

@Christoph-Hart looking forward to jumping on the MCP workflow!

[dannytaurus]

@Christoph-Hart said in New LAF Properties for Modulation Display: Selected Source vs Accumulated Range:

that's precisely the difference between "lol AI cannot do s***" and "boy we're f***ed"....

1,000% this

[David Healey]

@DanH said in New LAF Properties for Modulation Display: Selected Source vs Accumulated Range:

@Christoph-Hart on web for now....

I'm using it on the web too. Are you using Claude Code? You can pass it the link to your git repo

[dannytaurus]

@David-Healey HISE master - do you know something I don't?

[David Healey]

@dannytaurus My fork

[DanH]

@David-Healey setting it up now, have you across the AGENTS.md file at all?

[David Healey]

@DanH Nope

[DanH]

@Christoph-Hart let me see if I can course correct this PR

[DanH]

@Christoph-Hart PR sent, I'm not being precious about it so feel free to close again, good to get the practice in