[SoC] New rules for the Topology Framework in gvSIG Desktop - Reporte semanal - semana 2

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[SoC] New rules for the Topology Framework in gvSIG Desktop - Reporte semanal - semana 2

Mauro Carlevaro
Hola comunidad, envío el reporte semanal correspondiente al periodo del 3 al
9 de Junio.
    Qué pude completar esta semana?
En esta semana pude hacer las siguientes tareas:
       * Testear y depurar el código desarrollado.
       * Avanzar en la integración de la regla Must be coincident with con
el framework de topología.

    Qué voy a hacer la próxima semana?
       * Terminar la integración de la regla con el framework. Estimo que
para el 9th ya estará terminada
       * Hacer la regla: Must be covered by endpoint of
       * Testear y depurar el código desarrollado
       * Continuar documentando

    Hay algún problema, bloqueo?
    No hay ningún problema, ni bloqueo. Sin embargo, hacer la implementación
me ha llevado más tiempo del que pensé inicialmente, pero al hacer la
primera integración era previsible que lleve más tiempo. Para no atrasar los
demás sprints se planea dedicar más horas en el fin de semana.

Referencias:
Guía base desarrollada por Óscar Martínez para realizar las reglas y la
integración con el módulo de topología. Topology Rules for gvSIG Desktop:
Development guide. Link:
https://docs.google.com/document/d/1th0CVKOr918ltBI9czTJsRKAYSGnY6xpiY1R2415QMU)

TopologyRuleMustBeCoincidentWithPoint código de la integración link:
https://github.com/Maureque/TopologyRuleMustBeCoincidentWithPoint

GiitHub wiki link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/New-rules-for-the-Topology-Framework-in-gvSIG-Desktop
OSGeo wiki link:
https://wiki.osgeo.org/wiki/New_rules_for_the_Topology_Framework_in_gvSIG_Desktop

Cordiales saludos,
Mauro Carlevaro.



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html
_______________________________________________
gvSIG_desarrolladores mailing list
[hidden email]
Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: https://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores
Reply | Threaded
Open this post in threaded view
|

Re: [SoC] New rules for the Topology Framework in gvSIG Desktop - Reporte semanal - semana 2

Joaquin Jose del Cerro Murciano


El vie., 7 jun. 2019 a las 14:50, Mauro Carlevaro (<[hidden email]>) escribió:
Hola comunidad, envío el reporte semanal correspondiente al periodo del 3 al
9 de Junio.
    Qué pude completar esta semana?
En esta semana pude hacer las siguientes tareas:
       * Testear y depurar el código desarrollado.
       * Avanzar en la integración de la regla Must be coincident with con
el framework de topología.

    Qué voy a hacer la próxima semana?
       * Terminar la integración de la regla con el framework. Estimo que
para el 9th ya estará terminada
       * Hacer la regla: Must be covered by endpoint of
       * Testear y depurar el código desarrollado
       * Continuar documentando

    Hay algún problema, bloqueo?
    No hay ningún problema, ni bloqueo. Sin embargo, hacer la implementación
me ha llevado más tiempo del que pensé inicialmente, pero al hacer la
primera integración era previsible que lleve más tiempo. Para no atrasar los
demás sprints se planea dedicar más horas en el fin de semana.


Hola Mauro,
he echado un vistazo a lo que estas haciendo y te dejo
aqui algunos comentarios:

En la pagina "Must be coincident with" del wiki "gvsig-gsoc2019-topology" [1]
Comentas los pasos que has seguido en el desarrollo de la regla; pero no veo
donde describes lo que has hecho u obtenido en ellos. Por ejemplo... "Algorithm
design to solve the problema"... ¿ donde explicas cual va a ser o ha sido la
aproximacion a la solucion del problema ?
- Que datos de entrada tienes
- Como lo vas a abordar
- Que vas a obtener como salida (lo que va a ir al reporte de errores)
- Que acciones vas a implementar para corregir los errores
No se si no lo encuentro o es que aun no lo has puesto.

Por otro lado algunos comentarios sobre el codigo.
Hay algunos detalles que harian que la regla fallase durante la ejecucion, bueno,
probablemente no llegaria ni a ejecutarse; pero supongo que es que te falto algo
por subir a GitHub. Dejando esas cosas a porte, te comento sobre la clase
"MustBeCoincidentWithPointRule" [2].

- Estas usando un punto cuando llamas al query y otro (con un buffer) cuando
  llamas a instersects.
  Probablementet cuando llamas al query deberias usar tambien el buffer
  del punto.
 
- Cuando construyes la geometria que identifica al error estas haciendo
  un disjoin entre el punto con buffer y sin el... uff, no se si eso
  es lo adecuado... piensalo, probablemente que pongas el punto (sin buffer)
  sea mas adecuado.

- Echa un vistazo a las lineas 92 y siguientes...
  Si ese codigo llega a ejecutarse se producira un error tal cual esta.
  Mi consejo es que cambies:

    if theDataSet2.getSpatialIndex() != None:
 
  Por algo como:
 
    if False: #theDataSet2.getSpatialIndex() != None:  
 
  Para asi probar la ejecucion del else.

 
Comentar tambien sobre por que el "if" ese de si dispongo de indice espacial.
Normalmente si estamos trabajando con un shape, se creara el indice espacial
de forma automatica y se rellenara cuando vallamos a usarlo de forma implicita o
a traves del metodo "query" del DataSet. El problema es que si en lugar de
estar usando un shape, estas usando una tabla de BBDD (H2, postgres,...) no
se crearan indices espaciales en local. Es cuestion de que esten definidos
en la propia BBDD, y entonces, simplemente componiendo el filtro adecuado,
normalmente con el "expressionBuilder", ya los usara la BBDD. Si solo implementamos
la solucion usando el "query", la regla no ira con tablas de BBDD.

Otra cosa mas, en el fichero json, has dejado acciones vacias... mejor que definas
solo las acciones que toquen y no dejes entradas vacias que en un momento dado
pueden quedarse ahi y fallar mas adelante.



[1] https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/Must-be-coincident-with
[2] https://github.com/Maureque/TopologyRuleMustBeCoincidentWithPoint/blob/master/mustBeCoincidentWithPointRule.py

Un saludo
Joaquin
 
Referencias:
Guía base desarrollada por Óscar Martínez para realizar las reglas y la
integración con el módulo de topología. Topology Rules for gvSIG Desktop:
Development guide. Link:
https://docs.google.com/document/d/1th0CVKOr918ltBI9czTJsRKAYSGnY6xpiY1R2415QMU)

TopologyRuleMustBeCoincidentWithPoint código de la integración link:
https://github.com/Maureque/TopologyRuleMustBeCoincidentWithPoint

GiitHub wiki link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/New-rules-for-the-Topology-Framework-in-gvSIG-Desktop
OSGeo wiki link:
https://wiki.osgeo.org/wiki/New_rules_for_the_Topology_Framework_in_gvSIG_Desktop

Cordiales saludos,
Mauro Carlevaro.



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html
_______________________________________________
gvSIG_desarrolladores mailing list
[hidden email]
Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: https://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores


--
--------------------------------------
Joaquin Jose del Cerro Murciano
Development and software arquitecture manager at gvSIG Team
[hidden email]
gvSIG Association
www.gvsig.com

_______________________________________________
gvSIG_desarrolladores mailing list
[hidden email]
Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: https://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores
Reply | Threaded
Open this post in threaded view
|

Re: [SoC] New rules for the Topology Framework in gvSIG Desktop - Reporte semanal - semana 2

Mauro Carlevaro
Joaquin, muchas gracias por todas las aclaraciones, ahora sigo trabajando en
la integración pero voy a ponerme a trabajar en todos los puntos que
mencionas y envío un reporte.
Muchas gracias!!!!
Saludos
Mauro



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html
_______________________________________________
gvSIG_desarrolladores mailing list
[hidden email]
Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: https://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores