basicOverlaps is pretty weak
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza en el punto de entrada basicOverlaps y sigue el manejo de MV_3 descrito en el issue. Reproduce el ejemplo de roll y, después, inspecciona cómo la reutilización propuesta de zip podría depender de la detección de solapamientos; el trabajo terminado debe incluir un comportamiento decidido y cobertura para el caso de un subarray no correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The current basicOverlaps check doesn't actually fully determine if each of the constituent sub-arrays overlap.
e.g.
basicOverlaps (MV_3 n_1 as1 bs1 cs1) (MV_3 n_2 as2 bs2 cs2)
= M.basicOverlaps as1 as2
|| M.basicOverlaps bs1 bs2
|| M.basicOverlaps cs1 cs2
only checks the overlap between corresponding elements, but if you used something like
roll (MV_3 n as bs cs) = MV_3 n bs cs as
then something like overlaps mv (roll mv) will report False even though it has arrays that 'overlap' the originals rendering destructive changes hazardous.
I'm not really sure that this is such a damning failing, none of the existing combinators will produce such a rotation, but I figured it was worth capturing in an issue rather than passively ignoring the concern.
An example of where it might matter would be if you let zip for unboxed vectors try to get clever about reusing the source vectors.
- Lenguaje dominante
- Haskell
- Estrellas
- 403
- Forks
- 146
- Merge medio
- 1 d 22 h
- PR fusionados (30 d)
- 3
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de haskell/vector
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 50/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
`Size` can be a newtype.Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
Todos los issues de haskell/vector
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
halogenandtoast/ArkhamHorror#5825 · 1 comentario ·
-
docs: install-manifest download links use main instead of master (404)Posiblemente ocupada @ChinmayaBisoi la tomó hace 1 día. Abierto
Dificultad 2/5 Menos de una hora Aptitud para principiantes 78/100
hasura/graphql-engine#10884 ·
-
New-pipeline: update TracyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
AccelerateHS/accelerate#583 · 2 comentarios ·
-
component: hls-refactor-plugin status: needs triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 60/100
haskell/haskell-language-server#5111 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
alunduil/network-arbitrary#193 ·
Los mantenedores suelen responder en 1 día