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

Opened 14 years ago

Closed 14 years ago

#1080 closed enhancement (invalid)

TendLocPosTimeSet y los fechados que usa

Reported by: Jorge Conde Owned by: Víctor de Buen Remiro
Priority: low Milestone:
Component: ASCII data files Version: 2.0.1
Severity: minor Keywords: TendLocPosTimeSet, TendLocNegTimeSet, Normaliza, stdlib
Cc: Jorge, Pedro Gea

Description

Las funciones TendLocPosTimeSet, TendLocNegTimeSet y Normaliza, situadas en tolbase-v2.0.1\bin\stdlib\general\grammars\_serie.tol (al menos estas que haya comprobado, tal vez haya más en el fichero), generan solamente series en fechado diario: en su interior se fija el timeset de salida a "Diario".

Esto, si bien no es un problema, no está indicado en ningún lado, se debería incluir tal hecho en la descripción de la función (y tal vez en el nombre, aunque esto puede ser demasiado engorroso).

Lo que parece más lógico, es generalizar tales funciones para que la serie de salida pueda salir en cualquier fechado, esto es, añadir un argumento más que indique tal propiedad. Esto provoca un nuevo problema, y es que al tener dos fechados hay que tener en cuenta que sean compatibles (es decir, que el fechado que genera la tendencia esté contenido en el que genera la serie), pero si esto no ocurre tal vez se pueda dejar una comprobación inicial que de un error si esto no ocurre.

Change History (2)

comment:1 Changed 14 years ago by Víctor de Buen Remiro

Cc: Jorge Pedro Gea added

Estas funciones provienen de SADD y no tengo ni idea de quién las hizo.

Por otra parte no es posible añadir argumentos opcionales a una función de usuario así que no se pueden modificar sin provocar incompatibilidad hacia atrás, así que la única cosa que se puede hacer es crear nuevas funciones con otros nombres.

Lo que ya no tengo tan claro es el lugar donde crearlas. La idea es que StdLib se descomponga en paquetes en un futuro por lo que el ir engordándola no sé si es buena idea. Es algo que hay que discutir entre todos.

comment:2 Changed 14 years ago by Víctor de Buen Remiro

Resolution: invalid
Status: newclosed
Note: See TracTickets for help on using tickets.