Skip to content

Comments

fasttext: Bump version.#18

Merged
jlepere-everlaw merged 2 commits intomainfrom
feature/bump-version
Feb 23, 2026
Merged

fasttext: Bump version.#18
jlepere-everlaw merged 2 commits intomainfrom
feature/bump-version

Conversation

@jlepere-everlaw
Copy link

@jlepere-everlaw jlepere-everlaw commented Feb 23, 2026

This PR makes some changes to fasttext to support numpy>=2.0.

@jlepere-everlaw jlepere-everlaw requested a review from a team as a code owner February 23, 2026 19:27
This fixed errors like:

```
E           ValueError: Unable to avoid copy while creating an array as requested.
E           If using `np.array(obj, copy=False)` replace it with `np.asarray(obj)` to allow a copy when needed (no behavior change in NumPy 1.x).
E           For more details, see https://numpy.org/devdocs/numpy_2_0_migration_guide.html#adapting-to-changes-in-the-copy-keyword.
```

See [1].

[1] Everlaw/servers#49616 (comment)
The hope is that this'll build a new wheel with a later version of
`numpy`. See [1] for more details.

[1] Everlaw/servers#49616 (comment)
Copy link
Collaborator

@kmurphy4 kmurphy4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jlepere-everlaw jlepere-everlaw merged commit ad55a3d into main Feb 23, 2026
2 checks passed
@jlepere-everlaw
Copy link
Author

TYVM, @kmurphy4! Do you know if we have an idea out there about replacing fasttext? I thought there might have been one, but I couldn't find anything. I was thinking about pointing to this issue as a reason to prioritize a maintaned tool.

@jlepere-everlaw jlepere-everlaw deleted the feature/bump-version branch February 23, 2026 22:21
@kmurphy4
Copy link
Collaborator

TYVM, @kmurphy4! Do you know if we have an idea out there about replacing fasttext? I thought there might have been one, but I couldn't find anything. I was thinking about pointing to this issue as a reason to prioritize a maintaned tool.

I forwarded you an email chain from 12+ months ago that has some more context.

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.

2 participants