Possible typo in the stripInvalidFaceWorker() function
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- computer-graphics
Línea de trabajo
Open src/surface/simple_polygon_mesh.cpp around lines 548-553 and inspect SimplePolygonMesh::stripInvalidFaceWorker(). Verify whether the duplicated removeWithBadInds use should instead reference removeLowDegree; done means the unused parameter is handled correctly without changing the function's intended behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I noticed a possible typo in the function SimplePolygonMesh::stripInvalidFaceWorker() where the parameter removeLowDegree is unused (meanwhile removeWithBadInds is doubly used).
I think, on Line 549, that should be removeLowDegree instead of removeWithBadInds.
I don't believe this is causing any real issues. So, it' really a very minor type—just thought I’d point it out! Thanks for all your hard work in building this library!
- Lenguaje dominante
- C++
- Estrellas
- 1.3k
- Forks
- 183
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 nmwsharp/geometry-central
-
warning: control reaches end of non-void function in VectorHeatMethodSolver::computeLogMapPosiblemente ocupada @dyollb la tomó hace 63 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
nmwsharp/geometry-central#256 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
nmwsharp/geometry-central#258 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
nmwsharp/geometry-central#255 · 1 comentario ·
-
FlipEdgeNetwork::iterativeShorten() hangs on a contractible closed loopPosiblemente ocupada @designbynumbers la tomó hace 114 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
nmwsharp/geometry-central#253 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 57/100
nmwsharp/geometry-central#247 ·
Todos los issues de nmwsharp/geometry-central
Issues similares
-
chore: update release scheduleAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 4 días
-
nth_root does not restore the rounding mode and can return an upper bound below the true rootAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
scanaislop/aislop#482 ·
Los mantenedores suelen responder en 1 día
-
1 type: problem 2 scope: firmware 3 good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
SpaceTeam/STS1_COBC_SW#511 ·
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
imaaiagent/fumi#3 ·