Skip to content

New nodes: Logical boolean operations (OR, AND, XOR, NOT)#1399

Merged
Keavon merged 3 commits into
masterfrom
logic-ops
Aug 30, 2023
Merged

New nodes: Logical boolean operations (OR, AND, XOR, NOT)#1399
Keavon merged 3 commits into
masterfrom
logic-ops

Conversation

@isiko

@isiko isiko commented Aug 24, 2023

Copy link
Copy Markdown
Contributor

Adds these boolean operator nodes:

  • Not
  • And
  • OR
  • XOR

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 24, 2023 14:52 Inactive

@0HyperCube 0HyperCube left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - just some minor queries.

Comment thread node-graph/gcore/src/logic.rs Outdated
Comment thread node-graph/gcore/src/logic.rs Outdated
@Keavon Keavon changed the title Add boolean operations nodes New nodes: Logical boolean operations (OR, AND, XOR, NOT) Aug 25, 2023
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 26, 2023 18:51 Inactive
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 26, 2023 19:17 Inactive
@isiko

isiko commented Aug 26, 2023

Copy link
Copy Markdown
Contributor Author

Should be all fixed now :)

@0HyperCube

Copy link
Copy Markdown
Contributor

The "Not" node has, in the properties "Operand B" where there should be nothing.
Toggling "Operand B" using the expose dot for the other nodes actually toggles the first input.

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 29, 2023 18:20 Inactive
@Keavon Keavon force-pushed the master branch 16 times, most recently from 5328ea5 to 52b6a8d Compare August 30, 2023 00:59
@Keavon Keavon force-pushed the master branch 9 times, most recently from a0bf5c4 to 5944186 Compare August 30, 2023 02:37
isiko added 2 commits August 29, 2023 22:15
Adds OR AND XOR and NOT operators
Inputs to the 'Min' 'Max' 'Equality' and boolean operations nodes were
previously named 'First' and 'Second' and are now called 'Operand A' and
'Operand B'
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 30, 2023 06:02 Inactive
@Keavon Keavon merged commit 3419e73 into master Aug 30, 2023
@Keavon Keavon deleted the logic-ops branch August 30, 2023 06:55
@Keavon

Keavon commented Jun 13, 2026

Copy link
Copy Markdown
Member

Hi @isiko, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!

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.

3 participants