CpuMath Enhancement: Make bound checking of loops in hardware intrinsics more efficient
Los mantenedores suelen responder en 3 días
Nadie ha tomado este issue todavía.
- #2939 de @llRandom — cerrado sin fusionar
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- csharp
- Área
- performance
Línea de trabajo
Comienza leyendo las comprobaciones de los límites de los bucles en src\Microsoft.ML.CpuMath\SseIntrinsics.cs y src\Microsoft.ML.CpuMath\AvxIntrinsics.cs, junto con el issue vinculado y la discusión previa del pull request para entender el contexto. Actualiza las comprobaciones especificadas de forma coherente en ambos archivos y verifica que los bucles resultantes mantengan sus límites existentes mientras eliminan la instrucción innecesaria.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Style changes needed to solve part of https://github.com/dotnet/machinelearning/issues/823
Details
- In
src\Microsoft.ML.CpuMath\SseIntrinsics.csandsrc\Microsoft.ML.CpuMath\AvxIntrinsics.cs, changingwhile (pDstCurrent + 4 <= pDstEnd)for the loop bound checking intowhile (pDstCurrent <= pDstEnd - 4)to save an instruction (ref: https://github.com/dotnet/machinelearning/pull/668#issuecomment-412121893) - It may probably be a CoreCLR issue
- Lenguaje dominante
- C#
- Estrellas
- 9.4k
- Forks
- 2k
- Merge medio
- 3 d 3 h
- PR fusionados (30 d)
- 15
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 dotnet/machinelearning
-
LlamaTokenizer encodes some runs of repeated characters differently from SentencePiecePosiblemente ocupada @Laurianti la tomó hace 11 días. Abiertountriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
dotnet/machinelearning#7751 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
BertTokenizer merges words separated only by `\n`, `\t` or `\r`Posiblemente ocupada @rosebyte la tomó hace 19 días. Abiertoarea-Tokenizers green
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
dotnet/machinelearning#7724 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 3 días
-
Comment in example code not matching docstring for TimeSeriesCatalog.DetectSeasonality MethodAbiertoarea-TimeSeries documentation untriaged
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
dotnet/machinelearning#7205 ·
Los mantenedores suelen responder en 3 días
-
area-ONNX documentation Priority:3
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
dotnet/machinelearning#5894 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Add a .vsconfig fileAbiertoarea-Infrastructure Priority:2
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
dotnet/machinelearning#3260 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 3 días
Todos los issues de dotnet/machinelearning
Issues similares
-
bug next version
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Rem0o/FanControl.Releases#4220 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
nkdAgility/azure-devops-migration-tools#3217 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
skoruba/Duende.IdentityServer.Admin#335 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
question
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100