Use Ord by default for all set-like operations
Personne n'a encore pris cette issue.
- #203 par @JordanMartinez — fermée sans fusion
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Domaine
- backend
Piste de recherche
The issue names union, intersect, and difference, and points to #91 and #56 for prior decisions and behavior. Read those discussions and the existing implementations and tests for these functions to understand their current semantics and the proposed Ord default. Done means the intended complexity and duplicate behavior are agreed on and covered by tests; the long discussion and prior pull request show this work may need maintainer direction.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
We already made nub use Ord by default in #91, but there are a few other unnecessarily quadratic functions in this library still which probably deserve the same treatment. I think these are union, intersect, and difference.
As well as being quadratic, union and difference have a strange-seeming behaviour where duplicates are only preserved in the first argument:
union [0, 0] [1, 1] == [0, 0, 1]
See #56 for more info. We ought to be careful about changing this behaviour but I think we should consider it, since I think it’s very likely to trip people up and we will be making breaking changes to these functions anyway if we do this.
- Langage dominant
- PureScript
- Étoiles
- 58
- Forks
- 70
- Merge moyen
- 2 j 22 h
- PR mergées (30 j)
- 1
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 purescript/purescript-arrays
-
Description text of `difference`Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
purescript/purescript-arrays#246 ·
-
generate functionOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
purescript/purescript-arrays#244 · 2 commentaires ·
-
deleteWith function & friendsOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
purescript/purescript-arrays#242 · 2 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
purescript/purescript-arrays#241 ·
-
`groupBy` has unexpected `l` argument for `a -> a -> Boolean` predicatePeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 25/100
purescript/purescript-arrays#229 · 3 commentaires ·
Toutes les issues de purescript/purescript-arrays
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Mend: dependency security vulnerability
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Mend: dependency security vulnerability
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ManageIQ/manageiq-pods#1476 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
prime-radiant-inc/evener#6366 ·
Les mainteneurs répondent en général sous 1 jour