Skip to content

[ADD] Blog post providing book review of Animal Farm. - #25

Open
ultimatecoder wants to merge 4 commits into
masterfrom
master-book-review-animal-farm
Open

[ADD] Blog post providing book review of Animal Farm.#25
ultimatecoder wants to merge 4 commits into
masterfrom
master-book-review-animal-farm

Conversation

@ultimatecoder

Copy link
Copy Markdown
Owner
  • Description: A blog post describing my book review of the Animal Farm. Please write your comments here in the PR. I am expecting help with grammar and sentence composition. Thanks!

@ultimatecoder ultimatecoder added this to the January 2019 milestone Jan 4, 2019
@ultimatecoder
ultimatecoder force-pushed the master-book-review-animal-farm branch from 5cda202 to 571bb7a Compare January 8, 2019 12:16
@ultimatecoder
ultimatecoder force-pushed the master-book-review-animal-farm branch 3 times, most recently from 5a445fc to 1136b6a Compare January 26, 2019 10:45

@jesusinadidas jesusinadidas left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm on my phone and typing isn't much fun, so I just looked over the first section. Here are some ideas:

Animal Farm is one of the better books I've encountered [Encountered where? In my life? During my time as a student? Maybe add clarification here]. Written in 1945 by English author Eric Arthur Blair under his pen name George Orwell [I'd consider dropping this reference to his legal name and simply referring to him as "George Orwell", unless you're going to add a lot more information about his background], it is one of his best known and most influential works. [Any references to Orwell's reputation, be it for literary criticism, social commentary, satire, or his influences on modern popular culture and political culture would fit nicely here]

[Probably better to make this a new paragraph and elaborate a bit more on the significance of the following observations:]The sentences are well composed, which aided in my comprehension. The book consists of fewer than 100 pages and I was able to complete it in two sittings [consider omitting the comment on mileage variance, as this is common sense].

@ultimatecoder
ultimatecoder force-pushed the master-book-review-animal-farm branch from f7dc7b1 to 5133149 Compare January 27, 2019 13:36
@ultimatecoder ultimatecoder modified the milestones: January 2019, February Feb 1, 2019
ultimatecoder and others added 4 commits August 15, 2026 18:03
scripts/optimize-images made one attempt per photo — 1600px at quality 80 —
and accepted whatever came out. For a busy photo that is not enough. The
mural in the Animal Farm post settles at 403 KB, 96 KB over budget, and
re-encoding it does not help: passes two and three land on 403 KB again.

The script still printed its summary and exited 0, so the only signal was
scripts/test failing at --check, whose advice is to run this script. That
advice was a loop with no exit.

Photos now walk down 1600/1400/1200/1000px at quality 80 and keep the widest
size that fits. Each attempt re-encodes the untouched original, so a narrow
attempt never stacks its loss on a wider one. Narrowing is the right lever
here: 1200px/q80 holds edges that 1600px/q65 smears for about the same bytes.

Anything still over budget once the ladder bottoms out is now named on stderr
and exits non-zero, which covers GIFs too.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The photo went in at 4486x2982 straight off the camera, which no reader
needs and every feed reader renders at full size.

This is scripts/optimize-images output, not a hand encode: the ladder added
in the previous commit steps past 1600px and 1400px, both still over budget,
and settles at 1200px/q80. Re-running leaves the file byte-identical.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
jesusinadidas reviewed this in January 2019 and said plainly that they had
only read the first section. All four of their notes are taken:

  - "one of the good books I came across during 2017" said nothing about
    where. It now names the year as the field it is the best of.
  - Eric A. Blair is dropped. The post never uses the legal name again, so
    introducing it only slowed the first paragraph down.
  - Orwell's reputation gets the sentence they asked for, and the Times
    ranking now carries its year (2008).
  - The prose observation becomes its own paragraph and says why the
    plainness matters. The "your mileage may vary" aside is gone.

The other three sections were never reviewed and needed more work. Pings ->
Pigs, Coincidently -> Coincidentally, Old major -> Old Major. The plot
summary was one 34-line paragraph that switched between past and present
every few sentences; it is now four paragraphs in a consistent literary
present. Verb complementation ("succeed to out the owner"), articles ("take
the revenge", "as consequence"), and a handful of collocations ("inscribed a
deep impression on my soul", "improve in myself") are fixed throughout.

Two details about the book were wrong and are corrected: Snowball is driven
off the farm by Napoleon's dogs rather than assassinated, and Old Major dies
three nights after the meeting, not the next morning.

The image markdown was missing site.baseurl, so the src stayed relative in
production builds and feed readers had nothing to resolve it against. The
alt text now describes the mural instead of saying "title image".

Republished under today's date rather than the December 2017 one it was
written on, and the proofreader credit points at the reviewer.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ultimatecoder
ultimatecoder force-pushed the master-book-review-animal-farm branch from 5133149 to 0d34926 Compare August 15, 2026 12:41
@ultimatecoder

Copy link
Copy Markdown
Owner Author

Thanks for the review @jesusinadidas, and apologies that it took this long to come back to it. Every one of your notes is addressed below, and I have credited you as the proofreader on the post.

You mentioned you had only looked over the first section. That turned out to matter — the later sections needed more work than the first one, so I went through the whole post.

Your notes on the Introduction

Your note What changed
"Encountered where?" The sentence now names the field it is best of: "the best book I read in 2017."
Drop the reference to his legal name Eric A. Blair is gone. The post never used the name again, so introducing it only slowed the opening down.
Add something on Orwell's reputation Added a sentence on what he is remembered for, and the Times ranking now carries its year (2008).
Make this a new paragraph, elaborate on the significance The prose observation is its own paragraph and now says why the plainness matters rather than only that it is there.
Omit the mileage variance comment Removed. You were right that it was common sense.

The three sections you did not get to

  • Typos: PingsPigs, CoincidentlyCoincidentally, Old majorOld Major.
  • The plot summary was a single 34-line paragraph that switched between past and present every few sentences. It is now four paragraphs in a consistent literary present.
  • Verb complementation throughout: "succeed to out the owner", "supported them to create", "requested everyone to be allowed".
  • Articles: "take the revenge", "as consequence", "pay single attention".
  • Collocations: "inscribed a deep impression on my soul", "improve in myself", and "hegemony" used to mean the people holding power rather than the condition itself.

Two details about the book were also wrong and are corrected: Snowball is driven off the farm by Napoleon's dogs rather than assassinated, and Old Major dies three nights after the meeting rather than the next morning.

Rebase and CI

The branch is rebased onto current master. The repo has grown a CI pipeline since 2017, and the image budget added in #108 rejected this post's title photo, which went in at 4486x2982 and 3.4 MB.

Fixing that exposed a real bug in scripts/optimize-images: it made one attempt per photo at 1600px/q80 and accepted whatever came out. This photo settles at 403 KB, 96 KB over budget, and re-encoding does not help — passes two and three land on 403 KB again. The script still exited 0, so the only signal was scripts/test failing at --check, whose advice is to run the script. That advice was a loop with no exit.

Photos now walk down 1600/1400/1200/1000px and keep the widest size that fits, re-encoding from the untouched original each time so a narrow attempt never stacks its loss on a wider one. Anything still over budget once the ladder bottoms out is named on stderr and exits non-zero. The title photo is now 229 KB at 1200px, and it is the tool's output rather than a hand encode — re-running leaves it byte-identical.

Also fixed the image markdown, which was missing site.baseurl and left the src relative in production builds, so feed readers had nothing to resolve it against.

Verified locally before pushing: ./scripts/optimize-images --check passes, ./scripts/test passes with 179 external and 73 internal links checked, and a production build renders the image src as the absolute https://blog.jaysinh.dev/....

The post is republished under today's date rather than the December 2017 one it was written on.

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