Repository navigation
chore(register): C-62 — the pinned pipeline-core release still installs 2.5 GB of CUDA - #142
Merged
Merged
Conversation
…ls 2.5 GB of CUDA Maintainer challenge during the development->main sweep: "Is geopandas back? Is it still here?" Audited it properly. Source-level answer: no. The only three mentions of geopandas/shapely in .py or .toml are assertions of its absence (enrichment.py:9, build_gaul_lookup.py:11) plus a doc-accuracy test that bans the word. C-39's deletion held completely. Environment-level answer: yes. poetry.lock resolves geopandas 1.0.1, optional = false. The carrier is the pinned release — views-pipeline-core 2.3.0 declares geopandas, torch, scipy, seaborn, plotly and plotly-express. Their development branch already dropped geopandas and their own falsification tests name the problem; it is gated purely on publishing 3.0.0 (their #319/#313). Measured in the live project venv (2.8 GB total), none of it imported here: nvidia/ (torch CUDA runtime) 2.5 GB plotly 42 MB matplotlib 25 MB geopandas 1.6 MB The framing correction is recorded in the entry because it inverts the intuition that started the audit: geopandas is the architectural violation but is 1.6 MB. The material cost is torch's CUDA stack at 89% of the venv, in a repo with no GPU code, no training and no tensor operations. Optimising for the offensive dependency rather than the expensive one would have missed nearly the whole bill. Registered Tier 3 and deliberately NOT folded into C-44, because the two pull in opposite directions: C-44 holds the 3.0.0 bump on the maintainer's standing cross-repo constraint, C-62 records what waiting costs. Folding this into the entry arguing for the delay would hide the bill inside the argument. Also filed today: the geopandas claim in views-datafactory#387 was wrong and is corrected there — that repo does its spatial work geopandas-free already (shapely STRtree + pyproj, ADR-030), so the proposed equal-area check must not reintroduce it. And a consumer data point on pipeline-core#319. Register: 62 concerns, 25 open. Cluster G. Integrity guard green. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Answers the maintainer's question during the development→main sweep: "Is geopandas back? Is it still here?"
Short answer
Not in the code. Yes in the environment. And the real cost isn't geopandas.
Source level — clean
Three mentions of
geopandas/shapelyin.py/.toml, all asserting its absence:C-39's deletion held completely.
Environment level — not clean
poetry.lockresolvesgeopandas 1.0.1,optional = false. The carrier is the pinned release:Their
developmentalready dropped geopandas and their own falsification tests name the problem. It's gated purely on publishing 3.0.0.Measured, and it inverts the premise
Live project venv — 2.8 GB, none of it imported here:
nvidia/(torch CUDA runtime)plotlymatplotlibgeopandasThe audit started as "is geopandas back?" — and geopandas is the architectural violation, the thing PR #42 spent a 3,171-line deletion removing. But it's 1.6 MB. The material cost is torch's CUDA stack at 89% of the venv, in a repo with no GPU code, no training, no tensor operations.
Optimising for the offensive dependency rather than the expensive one would have missed nearly the whole bill. That correction is recorded in the entry, not just the commit.
Why C-62 and not an update to C-44
They pull in opposite directions, deliberately:
developmentacross all repos first. Still right.Folding this into the entry arguing for the delay would hide the bill inside the argument. Tier 3 — installed but never imported, so no correctness impact; the cost is footprint and a first release (#125) that sets expectations.
Cross-repo, filed today
pyproj, ADR-030 compliance). Left uncorrected it would have invited reintroducing it.Verification
Register integrity guard green. 62 concerns, 25 open. Cluster G. No source code touched.
🤖 Generated with Claude Code