Skip to content

Correct the last I-WLS claim, in Chapter 4's package docstring - #91

Merged
qmohsu merged 1 commit into
mainfrom
claude/ch4-init-docstring
Aug 23, 2026
Merged

Correct the last I-WLS claim, in Chapter 4's package docstring#91
qmohsu merged 1 commit into
mainfrom
claude/ch4-init-docstring

Conversation

@qmohsu

@qmohsu qmohsu commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

ch4_rf_point_positioning/__init__.py advertised "TOA/RSS positioning with I-WLS".

The example it describes solves with method="iterative_ls" at six of its seven call sites; the seventh, Example 6, supplies a covariance and is genuinely weighted. Same mislabel as the three files fixed in #87 and #90, one level up — found by re-sweeping after #90 merged.

This is the last one in the chapter: every remaining I-WLS under ch*/ is now in example_tdoa_positioning, at the two places a covariance really is passed.

Docstring only — no behaviour, no output, no figure. 83 passed across ch4, the lint ratchet and the sys.path guard; pyflakes clean.

🤖 Generated with Claude Code

`ch4_rf_point_positioning/__init__.py` advertised "TOA/RSS positioning with
I-WLS". The example it describes solves with `method="iterative_ls"` at six of
its seven call sites; the seventh, Example 6, supplies a covariance and is
genuinely weighted. Same mislabel as the three files fixed in #87 and #90, one
level up, and the last one in the chapter: every remaining "I-WLS" under `ch*/`
is now in `example_tdoa_positioning`, where a covariance really is passed.

Docstring only. No behaviour, no output, no figure.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@qmohsu
qmohsu merged commit e56ef92 into main Aug 23, 2026
1 check passed
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.

1 participant