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

Merge node: add execution mode and output for input index #2210

Merged
merged 3 commits into from
May 23, 2021

Conversation

MoritzBrueckner
Copy link
Collaborator

Closes #2189 and requires armory3d/armory_tools#8 for the documentation.

Node screenshot

There is now a dropdown that lets you choose whether the node should activate its output for each active input individually or only once per frame for the first active input. If the first mode is selected, there is a new output that lets you access the index of the input that activated the node.

@QuantumCoderQC
Copy link
Contributor

hey, great job!

I was also working on the exact same thing last night. I was trying to add node replacement feature to it as well, where I got stuck ;-)

@MoritzBrueckner
Copy link
Collaborator Author

Thanks, I hope I didn't invalidate too much of your work 😨 You're probably right that it needs node replacement, I didn't implement it because the default settings are still the same (no compatibility issues) but I guess it still needs an "Identity" replacement to automatically update the UI? I have to look this up every single time I work on it...^^

@MoritzBrueckner MoritzBrueckner marked this pull request as draft May 22, 2021 10:03
@MoritzBrueckner MoritzBrueckner marked this pull request as ready for review May 22, 2021 10:30
@luboslenco luboslenco merged commit 88f522c into armory3d:master May 23, 2021
@MoritzBrueckner MoritzBrueckner deleted the merge-node branch May 26, 2021 10:40
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.

[Feature request] Integer output from the merge node
3 participants