Change `and_` and `or_` to use `&` and `|` instead of `*` and `+` operators
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
Línea de trabajo
Empieza en openfisca_tools/model_api.py, en las líneas 75–77, e inspecciona cómo están definidos o aliasados and_, or_, add_ y multiply_. Confirma el comportamiento existente de los arrays booleanos y, después, haz que and_ y or_ usen los operadores solicitados explícitamente; el trabajo estará terminado cuando sus resultados no cambien y la implementación deje de estar duplicada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently, and_ and or_ are aliases for add_ and multiply_, respectively (or vice versa, they're duplicative). add_ and multiply_ apply + and * operators, respectively. I'd suggest that and_ and or_ instead apply & and | operators, respectively.
This won't change the result: np.array(bool) * np.array(bool) = np.array(bool), for example. But it would be more explicit, and could improve performance.
Relevant code:
https://github.com/PolicyEngine/openfisca-tools/blob/e2bc593b29f670f302f2c64b892a3df60b7cc30d/openfisca_tools/model_api.py#L75-L77
- Lenguaje dominante
- Python
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 11 h 16 min
- PR fusionados (30 d)
- 2
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 PolicyEngine/openfisca-tools
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
-
Partial formula executionAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
PolicyEngine/openfisca-tools#65 · 4 comentarios ·
-
Add piecewise formulasAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
Todos los issues de PolicyEngine/openfisca-tools
Issues similares
-
`burr` fails to start when the Python path has a space in itPosiblemente ocupada @enekoreto la tomó hoy. Abiertostatus/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 5 días
-
gemini 出现了新的域名需要加入Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
-
Dificultad 2/5 Medio día Aptitud para principiantes 72/100
debpalash/VoiceStudio#2903 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
dagster-io/community-integrations#367 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
aws-samples/sample-per-model-bedrock#243 ·
Los mantenedores suelen responder en 1 día