issue with missing values in Exercise 25 in chapter 5

Open

#526 opened on Feb 13, 2026 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 55/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Stale
Tech stack: r
Domain: documentation

Research direction

Open exercises/_05-ex-explore-numerical.qmd around lines 572-573 and inspect Exercise 25. Check how zero values are handled and render or preview the exercise; done means the transformation and accompanying wording consistently account for counties with zero values.

Written by the indexing model from the issue text.

Description

I suspect that the counties with 0 hispanic_2017 are getting silently elided when log(hispanic_2017) is run.

I wonder if the code https://github.com/OpenIntroStat/ims/blob/7227c6e2f90b9f1cb804aa013f40ae1eba41fae3/exercises/_05-ex-explore-numerical.qmd#L572C5-L573C47 might be updated to log(1 + hispanic_2017) and the text slightly revised: logs of one plus these values.

suppressPackageStartupMessages(library(openintro))
suppressPackageStartupMessages(library(tidyverse))
county_complete |>
  select(fips, name, state, hispanic_2017) |> 
  arrange(hispanic_2017) |>
  head()
#>    fips             name        state hispanic_2017
#> 1 28061    Jasper County  Mississippi             0
#> 2 30069 Petroleum County      Montana             0
#> 3 31005    Arthur County     Nebraska             0
#> 4 31009    Blaine County     Nebraska             0
#> 5 31115      Loup County     Nebraska             0
#> 6 46061    Hanson County South Dakota             0

^{Created on 2026-02-13 with reprex v2.1.1}

This doesn't directly relate to #525 except both are the same exercise.

Dominant language: JavaScript
Stars: 948
Forks: 211
PR merge metrics: No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[BUG] @mastra/inngest: errors thrown inside durable steps lose their original stack (and error type) — Inngest only sees the wrapper frame Open

bug customer-eng Durable Agents Inngest status: needs triage

Difficulty 2/5 1-3 hours Newbie friendliness 82/100

mastra-ai/mastra#24748 · 1 comment · Opened Sep 22, 2026
[optimization:agents-md-curator] Trim and correct root ambient context in github/gh-aw-firewall (LOGGING.md dead link, stale test claim, ~12% context reduction) Open

optimization optimization:agents-md-curator

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

githubnext/gh-aw-cao#13475 · Opened Sep 22, 2026
[BUG]: "Clear All" in Settings doesn't clear the saved analysis, old data comes back after reload Open

bug

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

AOSSIE-Org/OrgExplorer#253 · 1 comment · Opened Sep 22, 2026
isolated-declarations: TS9025 on an optional parameter property typed `readonly T[]`, which tsc 7 accepts Open

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

oxc-project/oxc#26944 · Opened Sep 22, 2026
bug report: ai observability - dry-running a paused evaluation emails the whole org Open

ai-observability bug team/ai-observability

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

PostHog/posthog#104687 · Opened Sep 22, 2026

More JavaScript issues