gvSIG bugs #4244

Error en el geoproceso "Comprobar coherencia de sentidos"

Added by Lluís Marqués almost 8 years ago. Updated almost 8 years ago.

Status:Closed% Done:

0%

Priority:NormalSpent time:-
Assignee:Francisco Díaz Carsí
Category:Geoprocess
Target version:2.3.0-2439 (rev. org.gvsig.desktop-2.0.145)
Severity:Minor Add-on version:
gvSIG version:2.3.0 Add-on build:
gvSIG build:2425 Add-on resolve version:
Operative System:Linux Add-on resolve build:
Keywords: Proyecto:
Has patch:Yes Hito:
Add-on name:Unknown

Description

Al ejecutar el geoproceso "Comprobar coherencia de sentidos" salta el siguiente mensaje de error:

es.unex.sextante.exceptions.WrongParameterIDException: Wrong parameter ID exception

Y la traza en el LOG es:

WARN [Thread-12] (SextanteLogHandler.java:37) - 
java.util.concurrent.ExecutionException: es.unex.sextante.exceptions.WrongParameterIDException: Wrong parameter ID exception
    at java.util.concurrent.FutureTask.report(FutureTask.java:122)
    at java.util.concurrent.FutureTask.get(FutureTask.java:188)
    at es.unex.sextante.gui.core.GeoAlgorithmExecutors$1.run(GeoAlgorithmExecutors.java:53)
    at java.lang.Thread.run(Thread.java:745)
Caused by: es.unex.sextante.exceptions.WrongParameterIDException: Wrong parameter ID exception
    at es.unex.sextante.core.ParametersSet.getParameter(ParametersSet.java:824)
    at es.unex.sextante.core.ParametersSet.getParameterValueAsInt(ParametersSet.java:886)
    at es.unex.sextante.topology.checkLineDirectionCoherence.CheckLineDirectionCoherenceAlgorithm.processAlgorithm(CheckLineDirectionCoherenceAlgorithm.java:76)
    at es.unex.sextante.core.GeoAlgorithm.execute(GeoAlgorithm.java:227)
    at es.unex.sextante.core.GeoAlgorithm.execute(GeoAlgorithm.java:173)
    at es.unex.sextante.core.ProcessTask.call(ProcessTask.java:59)
    at es.unex.sextante.core.ProcessTask.call(ProcessTask.java:13)
    at java.util.concurrent.FutureTask.run(FutureTask.java:262)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
    ... 1 more

checkLineDirectionPath.txt Magnifier (2.58 KB) Lluís Marqués, 06/24/2016 10:43 AM

Associated revisions

Revision 194
Added by Francisco Díaz Carsí almost 8 years ago

refs #4244: Fixed CheckLineDirectionCoherenceAlgorithm

Revision 871
Added by Francisco Díaz Carsí almost 8 years ago

refs #4244: Updated toolbox version

History

#1 Updated by Lluís Marqués almost 8 years ago

Al parecer el algoritmo espera tres parámetros y solo se pueden definir dos desde el interfaz de usuario. Falta un parámetro en el interfaz de usuario que permita definir un campo identificador de la capa de lineas seleccionada que permita agrupar fenómenos para ejecutar el algoritmo.

#2 Updated by Lluís Marqués almost 8 years ago

Añado parche.

#3 Updated by Daniel Martinez almost 8 years ago

  • Target version set to 2.3.0-2447-final (rev. org.gvsig.desktop-2.0.153)
  • Assignee set to Francisco Díaz Carsí

#4 Updated by Francisco Díaz Carsí almost 8 years ago

  • Status changed from New to Fixed

Tras aplicar el parche que adjunta Lluis (gracias Lluis) veo que el algoritmo tiene un error de bulto por el que no saca ninguna feature en la capa de resultado.

Lo arreglo y voy descubriendo otros errores que provocaban que no se añadieran todas las features de la capa original a la capa de resultado y que se añadieran muchas features duplicadas.

Los arreglo también.
Aún así, veo que este algoritmo tiene alguna laguna, por ejemplo, para saber que una linea tiene continuidad con otra, solo comprueba que los extremos están dentro de la tolerancia ¿y si los dos extremos están dentro de la tolerancia? No comprueba cuál de ellos es el más cercano con lo que intuyo que puede dar resultados no deseados.

En fin, creo que es un algoritmo que deberíamos revisar.

#5 Updated by Joaquín del Cerro Murciano almost 8 years ago

  • Target version changed from 2.3.0-2447-final (rev. org.gvsig.desktop-2.0.153) to 2.3.0-2439 (rev. org.gvsig.desktop-2.0.145)

#6 Updated by Álvaro Anguix almost 8 years ago

  • Status changed from Fixed to Closed

Also available in: Atom PDF