close Warning: Can't synchronize with repository "(default)" (/var/svn/mms does not appear to be a Subversion repository.). Look in the Trac log for more information.

Opened 14 years ago

Closed 13 years ago

#633 closed defect (fixed)

Problemas al eliminar un MElemento en una Jerarquía.

Reported by: livanr68 Owned by: Pedro Gea
Priority: critical Milestone: Release 0.6
Component: Models Keywords:
Cc:

Description

Trabajando en el ticket #453 me he encontrado que al eliminar un MElemento de una Jerarquía no se mantienen bien los coeficientes de los términos de la misma, te acompaño una secuencia de comandos que ilustran lo que te digo.

Con el ejemplo de matriculación de vehículos cargado ejecuto:

MMS::@Hierarchy hie = MMS::Container::GetModel(1)::GetHierarchy(2);
Matrix m1 = hie::GetHierarchyTerm(1)::GetCoefficients(?);
/*
4.0
4.0
6.0
2.0
3.0
1.0
*/
Real hie::RemoveMElement(2);
Matrix m2 = hie::GetHierarchyTerm(1)::GetCoefficients(?);
/*
4.0
1.0
6.0
2.0
3.0
*/

Sin embargo esta última matriz debería ser:

/*
4.0
6.0
2.0
3.0
1.0
*/

También he tenido problemas del mismo tipo al añadir nuevos MElementos, pues los valores de los coeficientes de los términos para estos nuevos MElementos no es 1 como debía sino que recupera valores anteriores, aunque me imagino que arreglando el problema de la eliminación se resuelva éste.

Change History (8)

comment:1 Changed 14 years ago by Pedro Gea

Status: newaccepted

El problema se extiende también a las sigmas relativas del ruido (noise) de la jerarquía.

comment:2 Changed 14 years ago by Pedro Gea

Resolution: fixed
Status: acceptedclosed

(In [2505]) Se modifica la gestión del argumento _.coefficients de los términos de la jerarquía, pasando a usar una matriz.
Se hace lo mismo para el argumento _.relativeSigmas del ruido de tipo normal.
Closes #633

comment:3 Changed 14 years ago by Pedro Gea

Resolution: fixed
Status: closedreopened

Los cambios crean nuevos errores cuando el ruido normal es de un submodelo.

comment:4 Changed 14 years ago by Pedro Gea

(In [2506]) Se añade el método _.GetNodeSize a los submodelos. Sin embargo el uso de este mecanismo adelanta la llamada a los datos.
Refs #633

comment:5 Changed 14 years ago by Pedro Gea

Priority: blockercritical

Es necesario revisar el diseño de la clase @Noise y sus derivadas. Habría que tener en cuenta:

  • la naturaleza de los datos si el noise está vinculado a un submodelo,
  • la posibilidad de establecerlos a través de una m-variable y
  • la posibilidad de establecer covarianzas (Véase #167)

comment:6 Changed 14 years ago by Pedro Gea

(In [2547]) El test 1C no pasa debido a los cambios concernientes al tique #633 ya que necesitan acceder a los datos en la definición.
Refs #633

comment:7 Changed 14 years ago by Pedro Gea

(In [2548]) Se corrige un bug en las sigmas relativas por defecto introducido en [2505].
Refs #633

comment:8 Changed 13 years ago by Pedro Gea

Resolution: fixed
sensitive: 0
Status: reopenedclosed
Note: See TracTickets for help on using tickets.