Closed LauraMA closed 11 years ago
Alguien cambió la función TwoReals
para que acepte argumentos que son los límites. Creo que es mejor que usen esto en test_intervalo.py
; deja el código más limpio. Espero su nueva implementación (ojo; han habido cambios que están en la nueva versión del repositorio).
No me quedó claro lo que no les queda claro del resultado de log()
. Yo diría que si hay valores negativos restrinja el dominio a la intersección con [0,inf](y que avise de esto solamente si hay una restricción del dominio) y en este caso, log(0)=-inf
; eso da consistencia con exp(-inf)=0
. Es este el problema?
Otra cosa: Por favor NO incluyan en sus commits los archivos .pyc
Corregí los tests de exp() y log() en test_intervalo.py
. Ahora pasan bien. El error era llamarlos usando np.exp(a) o np.log(a); lo correcto es usar a.exp() y a.log() para ejecutar las funciones correspondientes de intervalos.
Creo que hay un error si al restringir el dominio se obtiene un None, ya que de cualquier modo se pide que se calcule el log de restringido por eso agregue un try. Creo que no sé qué es lo que debe de regresar la función en este caso pero me pareció que debía ser un "nan".
El test considera intervalos positivos (aunque aun requiere un chequeo ) y funciona, pero no sé si debe considerar en el test intervalos arbitrarios (negativos) ya que eso se controla con la funcion log y su restringir dominio.
Lo que falta checar es que sea compatible con la función exp ya que al hacer muchos nosetests aún saca error. Por cierto también el test de la exp tiene este problema.