Two small JAX-related cleanups
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 52/100
Piste de recherche
Read src/array_api_extra/_lib/_helpers.py around the JAX capabilities() workaround and src/array_api_extra/_agnostic/_set.py around nunique. The issue leaves removal of the workaround contingent on deciding whether to support JAX versions below 0.6.0, so first check project policy and tests. Done means the comments accurately describe JAX's unique_counts behavior, and any shim or minimum-version documentation reflects the project's support decision.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
1. Drop the capabilities() shim for jax < 0.6.0?
_helpers.capabilities forces "boolean indexing" to False for JAX. This works around jax-ml/jax#27418, which was fixed in JAX 0.6.0 (March 2025). CI pins jax >= 0.10.2, so this branch never runs there (it is marked # pragma: no cover).
The package doesn't declare a minimum JAX version, so whether to remove it is a policy question: do we still want to support JAX < 0.6.0 at runtime? If not, the branch can be deleted:
out = xp.__array_namespace_info__().capabilities()
if _compat.is_torch_namespace(xp):
...
Removing it would also be a good moment to document a minimum supported JAX version.
2. Inaccurate comment in nunique
# 3. backend does not have unique_counts; e.g. wrapped JAX
JAX does have unique_counts. It takes the O(n log n) sort-based path because capabilities()["data-dependent shapes"] is False (the output shape of unique_counts can't be known under jax.jit). Also, JAX is no longer "wrapped": jax.numpy is used directly. Suggested wording:
# 3. backend lacks data-dependent shapes, so unique_counts is unusable
# (e.g. JAX, whose output shape can't be known under jax.jit)
The comment just below it, # xp does not have unique_counts; O(n*logn) complexity, should be updated the same way.
- Langage dominant
- Python
- Étoiles
- 33
- Forks
- 33
- Merge moyen
- 1 h 16 min
- PR mergées (30 j)
- 32
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de data-apis/array-api-extra
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
data-apis/array-api-extra#1005 · 15 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
ENH: new function `replace`Ouverteenhancement new function
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
data-apis/array-api-extra#992 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 56/100
data-apis/array-api-extra#1014 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
ENH, TST: Add support for MLX as array backendPeut-être pris @prady0t l’a pris il y a 4 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
data-apis/array-api-extra#1006 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
data-apis/array-api-extra#1002 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de data-apis/array-api-extra
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
SR_SECURITY_DESCRIPTOR.fromString drops the SACL when no DACL is presentPeut-être pris @paul7436 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
equinor/fmu-sumo-uploader#302 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
modelscope/evalscope#1821 ·
Les mainteneurs répondent en général sous 1 jour
-
Sanity on ansible-core devel fails: ignore-2.23.txt references the removed import-3.9 testPeut-être pris @yurnov l’a pris aujourd’hui. Ouverteneeds_triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 91/100
ansible-collections/kubernetes.core#1275 ·
Les mainteneurs répondent en général sous 1 jour