Repository navigation
fix: add missing quotes around species column in aes() exercise code - #69
Conversation
|
Hi! This PR fixes the missing quotes around |
|
Update (2026-10-22): I found a bug in my own commit — the added quotes inside the notebook JSON string were unescaped, which makes |
In data-visualise.ipynb the aes() calls passed the bare column name species, but plotnine requires column names to be quoted strings, so the snippet raised a NameError. Both aes(x = species) occurrences now read aes(x = "species"), with the inner quotes escaped so the notebook stays valid JSON. Signed-off-by: Jah-yee <166608075+Jah-yee@users.noreply.github.com>
91e9ec0 to
8bfe8ec
Compare
|
Hi aeturrell — checking back on the The change adds the missing quotes around the I also want to flag that my own follow-up commit earlier in this PR fixed an escaping mistake in the notebook JSON string (the added quotes had to be escaped inside the |
Summary
In the Data Visualisation chapter's exercises section, the example code uses which in Python's plotnine would attempt to reference a variable named rather than the column named .
Changed:
Files changed:
Affected lines (706, 709):
This is a single-commit PR with a minimal, surgical fix per the project guidelines.