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

Function Wizard: create Effects #1649

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

8-Lambda-8
Copy link
Contributor

Hi,
I added the possibility to create Effects with the Function Wizard
What types of effects arc creates is determined by the fixture capabilities (Dimmer, RGB, Position).
for each type a normal and a staggered version (start offset spread equally) will be created.

image
image
image

@mcallegari
Copy link
Owner

Hi, this doesn't build.
Can you please fix it?

@coveralls
Copy link

coveralls commented Jan 4, 2025

Coverage Status

coverage: 31.468% (-0.03%) from 31.502%
when pulling 1324743 on 8-Lambda-8:effectsInWizard
into 3ed321c on mcallegari:master.

@8-Lambda-8
Copy link
Contributor Author

i fixed it

i did not see that EFXFixture has a modeToString function

i removed the line and used this function instead

@yestalgia
Copy link
Contributor

Hey guys I've had a think about this.

Eventually I'd like to expand the EFX movement effect list. I think these should be separated in the wizard.

My suggestion would be:

  • Primary Colors
  • 16 Colors
  • Animations
  • EFX
    • Position
      • Line
      • Circle
      • etc.
    • RGB

Is there a better way for us to feed settings into the position effects like "Staggered", Split Reverse", "Asymmetrical" etc.?

@8-Lambda-8
Copy link
Contributor Author

Hi,
Yes, i will add the Modes (Pos, RGB, int) as sub items in the tree, but im not sure about the patterns. I think the list will get too long. And in my opinion you rarely want to create all of them, so you have to select/deselect to get your liked outcome. For me it would be enough to just create one and then manually duplicate and change the pattern. I see the real timesaver of this feature in generating the mode/offset/reverse settings.

For the offset/reverse settings, i could add columns with checkboxes (like odd/even)(i also have to implement odd/even)
What exactly do you mean with "Split Reverse" and "Asymetrical"? so i can add them as options

@8-Lambda-8
Copy link
Contributor Author

Hi, i added the mode sub items and the checkbox columns (not implemented the generation yet)

now it looks like this, what do you think?
image

@yestalgia
Copy link
Contributor

Mate we should be calling you the wizard! This is fantastic stuff!

What exactly do you mean with "Split Reverse" and "Asymetrical"? so i can add them as options

I'm referring to the fixture order setting in the EFX:
image

No stress if this is too complex.

im not sure about the patterns. I think the list will get too long.

Personally, I would find the patterns extremely helpful, but I understand the concern. Perhaps this would need to form part of a separate step.

@8-Lambda-8
Copy link
Contributor Author

8-Lambda-8 commented Mar 24, 2025

I'm referring to the fixture order setting in the EFX:

oh, i totally forgot that those exist, shouldn't be a problem to implement this.
and i realized that my "staggered" does the same as "Serial", so i will remove it.

also i noticed the "Serial" and "Asymmetric" are not displayed in the movement preview (all points are on the same spot), i will try to fix this and create a separate PR for that.

Personally, I would find the patterns extremely helpful, but I understand the concern. Perhaps this would need to form part of a separate step.

okay, i'll try to do that

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

Successfully merging this pull request may close these issues.

4 participants