Protected members (and @property)?
Los mantenedores suelen responder en 2 días
@dikwickley ya está trabajando en esto.
Desde el 13/2/2023.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- python
- Área
- backend-api-design
Línea de trabajo
Start by locating the linear-operator classes and reading where members are initialized and used by */matvec and .H/rmatvec. The issue does not name files or tests, and it leaves the scope of the naming and @property changes undecided; clarify which members should change before taking it on. Done would mean applying an agreed, consistent convention and verifying affected behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
So far the convention of prepending _ to highlight protected members is not used. While no operator requires the user to change anything after initialization and its use is limited to * (or matvec) and .H (or rmatvec), it may be better to be consistent with common practice.
Also, in some cases some variables are checked at initialization, it may more natural to convert those into @property. This is not just a pure stylistic change but may make better code (although will raise the bar for new developers?)
- Lenguaje dominante
- Python
- Estrellas
- 539
- Forks
- 126
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 21
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 PyLops/pylops
-
Lipschitz methodPosiblemente ocupada @Megumi1773 la tomó hace 28 días. Abiertoenhancement good first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Los mantenedores suelen responder en 2 días
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
PyLops/pylops#786 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Uv setup for intel environmentAbiertoCI
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 2 días
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Los mantenedores suelen responder en 2 días
-
Introduce spell checkingAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
PyLops/pylops#731 · 4 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
Todos los issues de PyLops/pylops
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
jejjohnson/gaussx#752 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
521xueweihan/HelloGitHub#4270 ·
-
enhancement owner: den-haag
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 6 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ok-oldking/ok-wuthering-waves#1828 ·
Los mantenedores suelen responder en 1 día
-
lesson-submission pending-review
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
Ikalus1988/MisakaNet#4512 ·
Los mantenedores suelen responder en 1 día