Skip to content

Fix and rephrase: 2.3.9, 2.3.41, 3.1.5, 3.1.21-23#44

Merged
spamegg1 merged 6 commits into
spamegg1:masterfrom
MetallicCrimson:fix_chapter_2_and_3
Dec 30, 2025
Merged

Fix and rephrase: 2.3.9, 2.3.41, 3.1.5, 3.1.21-23#44
spamegg1 merged 6 commits into
spamegg1:masterfrom
MetallicCrimson:fix_chapter_2_and_3

Conversation

@MetallicCrimson

@MetallicCrimson MetallicCrimson commented Dec 30, 2025

Copy link
Copy Markdown
Contributor

After I opened those discussions (each linked below), I thought I might as well just try and fix them myself.

I don't know anything about TeX, but editing the preexisting file was simple enough. (I hope I didn't break anything...) Still, I'm only studying these topics at the moment, so please check if my fixes are actually correct.

I applied the modifications mentioned in the following discussions, in chronological order, one commit at a time:

@spamegg1 spamegg1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good! Thanks a lot ❤️

@spamegg1 spamegg1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

All good

Comment thread src/Epp.tex
T & T & T & F & F & T & F & T & T \\
\hline
T & T & F & F & T & T & F & T & T \\
T & T & F & F & \color{cyan} T & T & \color{cyan} T & \color{cyan} T & \color{cyan} T \\

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Yep, second line is critical, not first.

Comment thread src/Epp.tex
\begin{tabular}{rrll}
(1) & & $p \to t$ & by premise (d) \\
& & ${\sim p}$ & by premise (c) \\ & $\therefore$ & ${\sim p}$ & by modus tollens \\
& & ${\sim t}$ & by premise (c) \\ & $\therefore$ & ${\sim p}$ & by modus tollens \\

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Yep, should be t instead of p.

Comment thread src/Epp.tex

\begin{proof}
The truth set is $\{-2, -1, 0, 1, 2\}$.
The truth set is $\{-2, -1, 1, 2\}$.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Right, 0 does not satisfy inequality.

Comment thread src/Epp.tex

\begin{proof}
$f$ is not differentiable, for some continuous function $f$.
$f$ is not differentiable for some continuous function $f$.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Yeah I tend to over-comma, English is my second language.

Comment thread src/Epp.tex

\begin{proof}
$\fa x$, if $x$ is a Java program, then $x$ has at least 5 lines.
$\fa$ program $x$, if $x$ is written in Java, then $x$ has at least 5 lines.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Makes sense, I didn't pay enough attention to the underlined placeholder in "for all ____ x".

Comment thread src/Epp.tex
$\fa x$ if $x$ is an equilateral triangle, then $x$ is isosceles.

$\fa$ equilateral triangles $x$, $x$ is isosceles.
$\fa x$, if $x$ is an equilateral triangle, then $x$ is isosceles.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Right, I guess I did the opposite thing in this and the previous ones! 😄

Comment thread src/Epp.tex
$\fa x$ if $x$ is a computer science student, then $x$ needs to take data structures.

$\fa$ computer science students $x$, $x$ needs to take data structures.
$\fa x$, if $x$ is a computer science student, then $x$ needs to take data structures.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

@spamegg1
spamegg1 merged commit 8c2f384 into spamegg1:master Dec 30, 2025
2 checks passed
@MetallicCrimson

Copy link
Copy Markdown
Contributor Author

No problem, and thank you for the merge.

However, the build failed at Release LaTeX document and source. Did I do something incorrectly?

(And Happy New Year btw!)

@spamegg1

Copy link
Copy Markdown
Owner

Did I do something incorrectly?

No, this is a recurring permission bug with Github, only happens on Release. I'll manually create a release now.

@spamegg1

Copy link
Copy Markdown
Owner

Happy New Year and thanks! 🎄

@MetallicCrimson
MetallicCrimson deleted the fix_chapter_2_and_3 branch January 2, 2026 12:02
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.

2 participants