IBCBiomech / insolesibc

Prototipo de aplicación para plantillas
0 stars 1 forks source link

Bug en selección de rangos #53

Open ivanjimenez opened 10 months ago

ivanjimenez commented 10 months ago

Al crear el GRF normalizado, el de las dos montañas, al seleccionar lo que hacemos es lo sabido, que utilizamos unos pasos que son los que comienzan en un rango, pero siempre cogiendo las montañas completas. Entonces hay un problema porque al seleccionar el rango verde, no da error como se ve en el primer caso. Pero en el segundo caso si utilizamos el rango rojo da un error.

El error tira del CalcularNormCurvas, al añadir las curvas. Te pongo más abajo una posible solución.

  1. Primer caso, seleccionando las señales en verde que están enteras, podemos sacar dos rangos.

Image

Image

  1. Segundo caso: si seleccionamos las señales en rojo da un error.

Image

Image

Revisa la función de normalización, creo que tiene que seleccionar según el rango elegido para crear las normalizaciones.

Una posible solución sería quitar los trozos en verde al principio y al final que no forman una montaña.

Image

bernatDiaz commented 10 months ago

MainWindow 06_11_2023 10_34_30 MainWindow 06_11_2023 10_35_25 Los rangos se ven muy diferentes depende de si seleccionas el verde o el rojo. Yo creo que el problema debe estar por allí.

private void RangeButton_Click(object sender, RoutedEventArgs e){
...
startL = xs_left_N_FC.IndexOf(Math.Round(XPoints[0] + 0.5, 2));
endL = xs_left_N_FC.IndexOf(Math.Round(XPoints[1] - 0.5, 2));
...
startR = xs_left_N_FC.IndexOf(Math.Round(XPoints[0], 2));
endR = xs_left_N_FC.IndexOf(Math.Round(XPoints[1], 2));

Esto tiene pinta de estar mal, deberia sumar y restar 0.5 a los dos o a ninguno

ivanjimenez commented 10 months ago

El range selected debe quedar siempre limpio. Aunque con un color haya por ejemplo más montanas que con otro color da igual, En el ejemplo que has hecho deberían quedar dos rojos y tres verdes. Y así ya normaliza bien. Realiza las pruebas que veas convenientes y luego te pego un toque.

bernatDiaz commented 10 months ago

Rama bugfix_seleccion_rangos he hecho algo parecido a esto: Una posible solución sería quitar los trozos en verde al principio y al final que no forman una montaña. He alargado el rango seleccionado por la izquierda y derecha hasta el medio del siguiente valle. Tambien se puede recortar hasta el medio del valle anterior.