Skip to content

Add logical operations: Nand and Nor. - #1332

Merged
rocky merged 1 commit into
masterfrom
Nand-and-Nor
May 1, 2021
Merged

Add logical operations: Nand and Nor.#1332
rocky merged 1 commit into
masterfrom
Nand-and-Nor

Conversation

@rocky

@rockyrocky commented May 1, 2021

Copy link
Copy Markdown
Member

Note: there will be futher changes to And when we merge ConditionalExpressions.

Note: there will be futher changes to And when we merge
ConditionalExpressions.
@rocky
rocky requested a review from mmateraMay 1, 2021 01:00
@rocky

rocky commented May 1, 2021

Copy link
Copy Markdown
MemberAuthor

@mmatera when this clears and gets into master, I will rebase #1330 for the further changes that are needed there. I can either put this in a new branch, or overwrite the existing one - your choice.

But the idea here, is that Nand and Nor is a separate useful thing independent of Condtional Expressions. By having these separated it helps better understand what's going on not just in that PR but in the future if we try to understand what's up or decouple the two.

Thanks for your patience and understanding.

@mmatera

Copy link
Copy Markdown
Contributor

@rocky, thanks for doing this. Go ahead with the merging and rebasing.

@rocky
rocky merged commit 5da8774 into masterMay 1, 2021
@rocky
rocky deleted the Nand-and-Nor branch May 1, 2021 04:44
@rocky
rocky restored the Nand-and-Nor branch May 1, 2021 04:44
@rocky
rocky deleted the Nand-and-Nor branch May 1, 2021 04:44
<dl>
<dt>'Nand[$expr1$, $expr2$, ...]'
<dt>'$expr1$ && $expr2$ && ...'
<dd> Implements the logical `NOR` function. The same that Not[And[$expr1$, $expr2$, ...]]

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

NOR seems to be a typo.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Thanks - should be fixed now in 369f6d1

Sign up for freeto 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

@rocky@mmatera@axkr