[Booleans] new concept #781

colinleach · 2024-09-09T18:28:23Z

For discussion on the forum.

This concept draft borrows heavily from exercises/concept.wip/.docs/introduction.md, with some edits and new additions. It pairs with annalyns-infiltration.

concepts/booleans/about.md

BNAndras · 2024-09-09T19:32:40Z

concepts/booleans/about.md

+true
+```
+
+Because `5 > 12` is `false`, clearly the combined expression must be `false`.


I don't believe we can say the combined expression is clearly false since we haven't introduced why the expression would be false. We also risk reinforcing whatever interpretation the student got from line 85 because they might think Julia stops when it hits a false value regardless of which operator is used so that's what "unambiguous" means.

BNAndras · 2024-09-09T19:34:35Z

concepts/booleans/about.md

+```
+
+Because `5 > 12` is `false`, clearly the combined expression must be `false`.
+The `1 != 0` expression is not relevant and is never evaluated.


1 != 0 isn't evaluated when the left side evaluates to false. It is evaluated if the left side evaluates to true so we can't say it's never evaluated without qualifying that never.

concepts/booleans/introduction.md

BNAndras · 2024-09-09T19:40:18Z

concepts/booleans/introduction.md

+true
+```
+
+Parentheses are optional and can make the code easier to read.


This should be included under the logical OR section. However, we've already mentioned this in the Boolean logic section so it can be dropped.

BNAndras · 2024-09-09T19:40:29Z

concepts/booleans/introduction.md

+
+In Julia (and many other programming languages), `&&` has a [higher precedence][operator-precedence] than `||` (in the same way that `*` is applied before `+`).
+This means that `true || false && true` evaluates to `true` because it is parsed as `(true || false) && true`.
+It is common to include explicit brackets anyway so that the reader doesn't need to think about this.


Suggested change

It is common to include explicit brackets anyway so that the reader doesn't need to think about this.

It is common to include explicit parentheses anyway so that the reader doesn't need to think about this.

Co-authored-by: András B Nagy <[email protected]>

colinleach · 2024-09-09T20:26:02Z

I need to think about this and try again.

Co-authored-by: András B Nagy <[email protected]>

colinleach · 2024-09-09T23:51:41Z

Your key point is that comparison operators aren't taught till the next concept. I'm a bit less worried about this in about.md, which is allowed to forward-reference, but these should definitely be removed from introduction.md.

One take-home message: don't blindly reuse text from the older exercise drafts, as I did here.. There are reasons why this syllabus hasn't launched.

I'll think about this, and try again tomorrow.

colinleach · 2024-09-20T17:45:19Z

I did a major rewrite (essentially from scratch) of both Markdown files, in response to the all-too-valid criticisms from @BNAndras.

Further comments welcome, from anyone.

BNAndras

Not at a computer but these jumped out at me.

concepts/booleans/about.md

BNAndras · 2024-09-20T18:44:10Z

concepts/booleans/about.md

+true
+```
+
+These operators will be familiar to users of many other languages, though *not* Python.


This sentence is confusing. and and or exist in other languages. Perhaps we say the operators and these concepts should be familiar coming from other languages but they might have different names there.

* Add `queen-attack` * Re-include tests * Add extra return at end of runtests.jl --------- Co-authored-by: depial <[email protected]>

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 4.2.1. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@692973e...eef6144) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [julia-actions/setup-julia](https://github.com/julia-actions/setup-julia) from 2.3.0 to 2.6.0. - [Release notes](https://github.com/julia-actions/setup-julia/releases) - [Changelog](https://github.com/julia-actions/setup-julia/blob/master/devdocs/making_a_new_release.md) - [Commits](julia-actions/setup-julia@780022b...9b79636) --- updated-dependencies: - dependency-name: julia-actions/setup-julia dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [julia-actions/cache](https://github.com/julia-actions/cache) from 2.0.4 to 2.0.5. - [Release notes](https://github.com/julia-actions/cache/releases) - [Changelog](https://github.com/julia-actions/cache/blob/main/devdocs/making_a_new_release.md) - [Commits](julia-actions/cache@8608484...8242439) --- updated-dependencies: - dependency-name: julia-actions/cache dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.4.0 to 4.4.3. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@5076954...b4b15b8) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.2.1 to 4.2.2. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@eef6144...11bd719) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [julia-actions/setup-julia](https://github.com/julia-actions/setup-julia) from 2.6.0 to 2.6.1. - [Release notes](https://github.com/julia-actions/setup-julia/releases) - [Changelog](https://github.com/julia-actions/setup-julia/blob/master/devdocs/making_a_new_release.md) - [Commits](julia-actions/setup-julia@9b79636...5c9647d) --- updated-dependencies: - dependency-name: julia-actions/setup-julia dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.4.3 to 4.5.0. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@b4b15b8...6f51ac0) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

… order of the return values (exercism#827)

Bumps [julia-actions/cache](https://github.com/julia-actions/cache) from 2.0.5 to 2.0.7. - [Release notes](https://github.com/julia-actions/cache/releases) - [Changelog](https://github.com/julia-actions/cache/blob/main/devdocs/making_a_new_release.md) - [Commits](julia-actions/cache@8242439...2b1bf4d) --- updated-dependencies: - dependency-name: julia-actions/cache dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.5.0 to 4.6.0. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@6f51ac0...65c4c4a) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* WIP draft of numbers concept * enlarged/corrected section on rounding * added section on isapprox() for floats --------- Co-authored-by: Colin Leach <[email protected]>

Co-authored-by: András B Nagy <[email protected]>

colinleach · 2025-02-26T22:23:50Z

Aargh! I have done something very, very wrong with this, and the files changed now include GH workflows and all sorts of nonsense.

Unless I can dig myself out of this soon, I may close this PR and open a new one. It was just about ready to review, before I broke it!

colinleach · 2025-02-26T22:56:02Z

Closing now, I wrecked this PR. Continues with #831.

colinleach added 2 commits September 8, 2024 16:20

new concept booleans

6cf0ac9

Merge branch 'exercism:main' into booleans

47921e7

BNAndras reviewed Sep 9, 2024

View reviewed changes

colinleach and others added 2 commits September 9, 2024 13:23

Update concepts/booleans/about.md

98d337d

Co-authored-by: András B Nagy <[email protected]>

Update concepts/booleans/about.md

e322b73

Co-authored-by: András B Nagy <[email protected]>

Update concepts/booleans/introduction.md

e08b195

Co-authored-by: András B Nagy <[email protected]>

colinleach and others added 2 commits September 10, 2024 07:49

parens

31026c3

Merge branch 'exercism:main' into booleans

46be2c5

colinleach mentioned this pull request Sep 20, 2024

Move old exercises/concept.wip drafts #789

Closed

colinleach added 2 commits September 20, 2024 10:42

rewrite of both about.md and introduction.md

202fac9

Merge branch 'main' into booleans

12396fd

BNAndras reviewed Sep 20, 2024

View reviewed changes

colinleach and others added 15 commits September 21, 2024 17:21

[Annalyns Infiltration] concept exercise (exercism#782)

a467cbb

Add strain (exercism#794)

cc5276c

Add queen-attack (exercism#798)

c8ca094

* Add `queen-attack` * Re-include tests * Add extra return at end of runtests.jl --------- Co-authored-by: depial <[email protected]>

Add proverb (exercism#807)

7aacda3

update CI versions to Julia 1.11 (current) and 1.10 LTS (exercism#809)

31319f9

exercises/practice/anagram: add an append with instructions about the…

fa278c7

… order of the return values (exercism#827)

colinleach and others added 11 commits February 26, 2025 12:38

WIP draft of numbers concept (exercism#792)

565e825

* WIP draft of numbers concept * enlarged/corrected section on rounding * added section on isapprox() for floats --------- Co-authored-by: Colin Leach <[email protected]>

Update concepts/booleans/about.md

11bc8e0

Co-authored-by: András B Nagy <[email protected]>

new concept booleans

c113a88

Update concepts/booleans/about.md

bd8fbf0

Co-authored-by: András B Nagy <[email protected]>

Update concepts/booleans/about.md

d295dce

Co-authored-by: András B Nagy <[email protected]>

Update concepts/booleans/introduction.md

96ca716

Co-authored-by: András B Nagy <[email protected]>

parens

0d71579

rewrite of both about.md and introduction.md

f142448

tweaked wording for clarity

a684d7c

reused old UUID

76ed0cc

Merge remote-tracking branch 'origin/booleans' into booleans

066339a

colinleach marked this pull request as draft February 26, 2025 22:21

colinleach mentioned this pull request Feb 26, 2025

[Booleans] another attempt #831

Merged

colinleach closed this Feb 26, 2025

colinleach deleted the booleans branch February 27, 2025 00:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[Booleans] new concept #781

[Booleans] new concept #781

Uh oh!

colinleach commented Sep 9, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

BNAndras Sep 9, 2024

Uh oh!

BNAndras Sep 9, 2024

Uh oh!

Uh oh!

BNAndras Sep 9, 2024

Uh oh!

BNAndras Sep 9, 2024

Uh oh!

colinleach commented Sep 9, 2024

Uh oh!

colinleach commented Sep 9, 2024 •

edited

Loading

Uh oh!

colinleach commented Sep 20, 2024

Uh oh!

BNAndras left a comment

Uh oh!

Uh oh!

BNAndras Sep 20, 2024

Uh oh!

colinleach commented Feb 26, 2025

Uh oh!

colinleach commented Feb 26, 2025

Uh oh!

Uh oh!

	It is common to include explicit brackets anyway so that the reader doesn't need to think about this.
	It is common to include explicit parentheses anyway so that the reader doesn't need to think about this.

Uh oh!

[Booleans] new concept #781

[Booleans] new concept #781

Uh oh!

Conversation

colinleach commented Sep 9, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

BNAndras Sep 9, 2024

Choose a reason for hiding this comment

Uh oh!

BNAndras Sep 9, 2024

Choose a reason for hiding this comment

Uh oh!

Uh oh!

BNAndras Sep 9, 2024

Choose a reason for hiding this comment

Uh oh!

BNAndras Sep 9, 2024

Choose a reason for hiding this comment

Uh oh!

colinleach commented Sep 9, 2024

Uh oh!

colinleach commented Sep 9, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

colinleach commented Sep 20, 2024

Uh oh!

BNAndras left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

BNAndras Sep 20, 2024

Choose a reason for hiding this comment

Uh oh!

colinleach commented Feb 26, 2025

Uh oh!

colinleach commented Feb 26, 2025

Uh oh!

Uh oh!

colinleach commented Sep 9, 2024 •

edited

Loading