From mauroctecno en gmail.com  Fri Jun  7 14:49:50 2019
From: mauroctecno en gmail.com (Mauro Carlevaro)
Date: Fri, 7 Jun 2019 05:49:50 -0700 (MST)
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology Framework
 in gvSIG Desktop - Reporte semanal - semana 2
Message-ID: <1559911790966-0.post@n6.nabble.com>

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

From jjdelcerro en gvsig.org  Sun Jun  9 20:11:32 2019
From: jjdelcerro en gvsig.org (Joaquin Jose del Cerro Murciano)
Date: Sun, 9 Jun 2019 20:11:32 +0200
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology
 Framework in gvSIG Desktop - Reporte semanal - semana 2
In-Reply-To: <1559911790966-0.post@n6.nabble.com>
References: <1559911790966-0.post@n6.nabble.com>
Message-ID: <CABLUYi1bTcXvFnfmqTqLZuH-=Dpz-jY_VtD6kCBsOG+Mfvn5sA@mail.gmail.com>

El vie., 7 jun. 2019 a las 14:50, Mauro Carlevaro (<mauroctecno en gmail.com>)
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
> gvSIG_desarrolladores en listserv.gva.es
> 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
jjdelcerro en gvsig.com
gvSIG Association
www.gvsig.com
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: <http://listserv.gva.es/pipermail/gvsig_desarrolladores/attachments/20190609/14e05893/attachment.html>

From mauroctecno en gmail.com  Sun Jun  9 22:29:51 2019
From: mauroctecno en gmail.com (Mauro Carlevaro)
Date: Sun, 9 Jun 2019 13:29:51 -0700 (MST)
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology
 Framework in gvSIG Desktop - Reporte semanal - semana 2
In-Reply-To: <CABLUYi1bTcXvFnfmqTqLZuH-=Dpz-jY_VtD6kCBsOG+Mfvn5sA@mail.gmail.com>
References: <1559911790966-0.post@n6.nabble.com>
	<CABLUYi1bTcXvFnfmqTqLZuH-=Dpz-jY_VtD6kCBsOG+Mfvn5sA@mail.gmail.com>
Message-ID: <1560112191447-0.post@n6.nabble.com>

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

From mauroctecno en gmail.com  Fri Jun 14 14:41:44 2019
From: mauroctecno en gmail.com (Mauro Carlevaro)
Date: Fri, 14 Jun 2019 05:41:44 -0700 (MST)
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology Framework
 in gvSIG Desktop- Reporte semana 3
Message-ID: <1560516104625-0.post@n6.nabble.com>

Hola,
Envío el reporte semanal correspondiente al periodo del 10 al 16 de Junio,
aún sigo trabajando en mejorar lo expuesto en la wiki pero quiero agradecer
a Joaquín por los comentarios y sugerencias, todos los aportes son de gran
ayuda para seguir mejorando.

Qué pude completar esta semana? 
* Hacer un análisis previo de la regla Must be covered by endpoint of antes
de la implementación siguiendo las pautas del documento realizado por Óscar
Martínez[1]
* Comenzar con la codificación de la regla Must be covered by endpoint[2] of
y dejar pronta la estructura de archivos necesaria para realizar la
integración con el marco de topología.[3]
* Testear y depurar el código desarrollado. 

 Qué voy a hacer la próxima semana? 
* Dejar pronta la regla y su integración con el marco de topología.
* Testear y depurar el código desarrollado. 
* Seguir documentando todo el proceso.

Hay algún problema, bloqueo?  No hay problema de bloqueo. La implementación
está siendo laboriosa pero se estima que el tiempo extra que llevo la
primera parte de la regla de esta semana, en comparación al empleado en el
desarrollo de la primera regla, se compensará al momento de hacer la
integración.
A pesar que las reglas van creciendo en cuanto a complejidad, el aprendizaje
de cada semana contribuye para poder cumplir a tiempo los objetivos.

Referencias:
[1].
https://docs.google.com/document/d/1th0CVKOr918ltBI9czTJsRKAYSGnY6xpiY1R2415QMU
[2].
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/4.-Must-be-covered-by-endpoint-of
[3]. https://github.com/Maureque/TopologyRuleMustBeCoveredByEndpointOfPoint

GiitHub wiki link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/1.-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

Saludos,
Mauro Carlevaro.



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html

From mauroctecno en gmail.com  Fri Jun 21 14:06:58 2019
From: mauroctecno en gmail.com (Mauro Carlevaro)
Date: Fri, 21 Jun 2019 05:06:58 -0700 (MST)
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology Framework
 in gvSIG Desktop- Reporte semana 4
Message-ID: <1561118818818-0.post@n6.nabble.com>

Hola, envío el reporte semanal correspondiente al periodo del 17 al 23 de
Junio.

Qué pude completar esta semana?
* Quedó implementada la regla Must be covered by endpoint of integrada con
el framework de topología. Se tomó en cuenta el caso de multinlineas.
* Testear y depurar el código desarrollado.
* Se continuó documentando todo el proceso.

Qué voy a hacer la próxima semana?
* Para la semana próxima además de estudiar la nueva regla a implementar
Points must be covered by line se agregara la consideración de que se tenga
multipuntos en la regla Must be coincident with, ya que, no va a fallar pero
puede dar resultados erroneos.
* Testear y depurar el código desarrollado.
* Seguir documentando todo el proceso.

Hay algún problema, bloqueo?  No hay problema de bloqueo. 

Referencias:
    Reporte semana 4. Link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/4b.-Report-Week-4-(June-17th-to-June-23rd)
    Regla Must be covered by endpoint of. Link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/4.-Must-be-covered-by-endpoint-of
    Wiki GitHub, link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki
    Wiki OSGeo, link:
https://wiki.osgeo.org/wiki/New_rules_for_the_Topology_Framework_in_gvSIG_Desktop

Saludos,
Mauro



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html

From mauroctecno en gmail.com  Fri Jun 28 15:31:37 2019
From: mauroctecno en gmail.com (Mauro Carlevaro)
Date: Fri, 28 Jun 2019 06:31:37 -0700 (MST)
Subject: [Gvsig_desarrolladores] [SoC] New rules for the Topology Framework
 in gvSIG Desktop- Reporte semana 5
Message-ID: <1561728697115-0.post@n6.nabble.com>

Hola, envío el reporte semanal correspondiente al periodo del 24 al 30 de
Junio.

Qué pude completar esta semana?
* Estudio de la regla Points must be covered by line
* Se agregó la consideración de que se tenga multipuntos en la regla Must be
coincident with.
* Desarrollo de la primera parte del código de la regla Points must be
covered by lin para la integración.
* Se continuó mejorando la documentación, se agrego una sección sobre el
plan de testing.

Qué voy a hacer la próxima semana?
* Realizar la integración de la regla Points must be covered by line con el
framework de topología.
* Optimizar el algoritmo desarrollado.
* Testear y depurar el código desarrollado.
* Seguir documentando todo el proceso. 

Hay algún problema, bloqueo?  No hay problema de bloqueo.

Referencias:
    Reporte semana 5. Link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/5a.-Report-Week-5-(June-24th-to-June-30th)
    Regla Points must be covered by line. Link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki/5.-Points-must-be-covered-by-line
    Wiki GitHub, link:
https://github.com/Maureque/gvsig-gsoc2019-topology/wiki
    Wiki OSGeo, link:
https://wiki.osgeo.org/wiki/New_rules_for_the_Topology_Framework_in_gvSIG_Desktop

Saludos,
Mauro 



--
Sent from: http://osgeo-org.1560.x6.nabble.com/gvSIG-desarrolladores-f4163512.html