Source of truth on parameter constraints unclear

Abierto

#293 abierto el 29/4/2025 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 4/5
Tiempo estimado: 3-5 días
Aptitud para principiantes: 38/100

Tipo de issue: Refactorización
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: python
Área: backend

Línea de trabajo

Comienza comparando las restricciones de los parámetros en src/openlifu/plan/protocol.py, en las líneas 68 y 74; después, sigue la ruta del protocolo y el uso de SolutionAnalysisOptions descritos en el issue. Confirma qué restricciones se aplican durante el análisis y aclara o consolida la fuente de verdad para que quienes creen protocolos tengan un único lugar inequívoco donde definirlas.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Parameter constraints are defined both in Protocol and in SolutionAnalysisOptions.
But it's not clear which is going to end up being used.
Currently it appears to be the one in protocol.
And as far as I can see the one in SolutionAnalysisOptions is not used, unless one manually calls solution.analyze and passes something different.

The issue is that when one creates a protocol it's not clear whether the place to define parameter constraints is here or here.

Lenguaje dominante: Python
Estrellas: 27
Forks: 21
Merge medio: 1 d 20 min
PR fusionados (30 d): 6

Guía de contribución

Abrir 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 OpenwaterHealth/openlifu-python

Add transducer_transform_source_id to SolutionInfo for source-liveness tracking Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100

OpenwaterHealth/openlifu-python#492 · Abierto el 27/7/2026
Add array_transform to SolutionInfo to preserve compute-time transducer pose Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

OpenwaterHealth/openlifu-python#491 · Abierto el 27/7/2026
Save Focal Gain LUT into a data file (json, csv, etc.) to make the source code file shorter. Abierto

enhancement

Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100

OpenwaterHealth/openlifu-python#448 · 2 comentarios · Abierto el 17/4/2026
RFC: Feedback abstraction for adaptive/closed-loop sanitation research Abierto

Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100

OpenwaterHealth/openlifu-python#498 · Abierto el 19/8/2026
Reconstruction quality is inconsistent +poor across runs on identical input + very slow processing Abierto

OpenwaterHealth/openlifu-python#496 · 7 comentarios · 2 asignados · Abierto el 7/8/2026

Todos los issues de OpenwaterHealth/openlifu-python

Issues similares

validate-structure inspecte challenge/work/.terraform et rend faux les labs qui ont un provider en cache Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

stephrobert/dsoxlab#238 · Abierto el 24/9/2026
Uniform probability densities use the upper bound as an interval width Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

nasa/fmdtools#40 · Abierto el 24/9/2026
_common_folder() treats a single file as the common folder; single-file archives can't be installed (4.2.8) / crash (master) Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

sublimehq/package_control#1780 · Abierto el 24/9/2026
Set 'Sample' input in 'autorun.py' to utilize 'Sample' Tab in eLog Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100

pcdshub/mfx#309 · 2 comentarios · Abierto el 24/9/2026
niri config file addition does not respect existing optional include Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

nwg-piotr/nwg-displays#145 · Abierto el 24/9/2026

Más issues de Python

Un resumen breve de issues de GitHub para principiantes.