Add validation of segments when writing a `MultiRecord`
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Jupyter Notebook
- Estrellas
- 853
- Forks
- 322
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
As mentioned in #555, when writing multisegment records we should validate the segments. A good initial step would be to simply add the already developed _check_segment_cohesion validation to the workflow.
There are likely a number of other validations that should be performed. For example, we may want to confirm that samps_per_frame is set for segments with expanded signals and that their length is as expected.
Please add any additional suggestions for validating the segments below.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Línea de trabajo
El issue no menciona ningún archivo ni prueba; empieza siguiendo el flujo de escritura de MultiRecord y localizando la validación existente _check_segment_cohesion. Usa #555 y la validación actual como contexto y confirma qué comprobaciones adicionales de segmentos están dentro del alcance y qué criterios de finalización deberían aplicarse.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- data
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100