Skip to content

docs(readme): note the 5.10.3 upgrade behaviour change - #93

Merged
Ravenwater merged 1 commit into
mainfrom
docs/readme-upgrade-note
Aug 28, 2026
Merged

docs(readme): note the 5.10.3 upgrade behaviour change#93
Ravenwater merged 1 commit into
mainfrom
docs/readme-upgrade-note

Conversation

@Ravenwater

Copy link
Copy Markdown
Contributor

The v5.10.3 release page now carries the full explanation of the .noconvert() change — but PyPI renders the README, not release notes. Someone running pip install -U mtl5 had no way to find out that vector()/matrix() can now raise on input that worked in 5.10.2.

Adds a short ### Upgrading to 5.10.3 under Install, where someone who just upgraded will actually look. It covers three things and links out for the rest:

  • who is affected — stated as concrete inputs (a non-contiguous array: a[::2], a column, a transpose; or a dtype outside the registered nine) rather than as a description of the change
  • the one-line fixnp.ascontiguousarray(a), framed as preserving precision rather than as a workaround, since that is what it does
  • why it is a fix, not a restriction — the old path silently downcast a float64 slice to float32 while reporting is_view=True and aliasing nothing

Docs only. Lands as docs:, which is not in patch_tags, so it cuts no release.

Worth dropping this section once 5.10.3 is a few releases back — noted in the commit message so whoever prunes the README knows it was meant to be temporary.

🤖 Generated with Claude Code

The v5.10.3 release page carries the full explanation, but PyPI shows the
README rather than release notes -- so a `pip install -U mtl5` user had no
way to see that vector()/matrix() can now raise on input that worked in
5.10.2.

Short version under Install, where someone who just upgraded will look:
who is affected (non-contiguous arrays, dtypes outside the registered
nine), the one-line fix, and why it is a fix rather than a restriction --
the old path silently downcast a float64 slice to float32 while
reporting is_view=True.

Links out to the release notes rather than repeating them. Worth dropping
once 5.10.3 is a few releases back.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

On-demand reviews are free for the next 23 days. After that, they cost $0.25 per reviewed file.

Or wait 49 minutes for your next included review.

View limit details

Limit details: You’ve used the included review currently available. Your 73 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dca5c274-ed4a-4f9d-9a34-3b2333cbb94f

📥 Commits

Reviewing files that changed from the base of the PR and between 997b5a0 and 36df1a8.

📒 Files selected for processing (1)
  • README.md

Comment @coderabbitai help to get the list of available commands.

@Ravenwater Ravenwater self-assigned this Aug 28, 2026
@Ravenwater
Ravenwater merged commit 91ee57c into main Aug 28, 2026
14 checks passed
@Ravenwater
Ravenwater deleted the docs/readme-upgrade-note branch August 28, 2026 08:58
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