-
Notifications
You must be signed in to change notification settings - Fork 8
Deber de comparacion del tiempo de ordenamiento entre bubble sort e i… #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Gracias José he revisado el aporte de algoritmos de ordenamiento. Sería interesante que una nueva versión tenga las funciones bubble sort, insertion sort y la de ingreso de tamaño del arreglo en otra carpeta para hacer uso de la programación modular, simplificando el programa principal más. También, podrías considerar en vez de usar input, utilizar la librería argparse para pasar como argumento desde terminal de la consola el tamaño del arreglo. Estas observaciones como mejora. De ahí todo se ve muy bien. Gracias por contribuir y trabajar. |
LeninGF
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excelente trabajo, toma en consideración las observaciones realizadas. Esta pendiente que ingreses el método de entrenamiento y el de predicción. Sin embargo, dado que no les he subido el código de ejemplo, es un muy buen inicio!!!!
| fpaso = [1. if elem >= 0 else 0. for elem in net] | ||
| return fpaso | ||
| def sigmoide(net): | ||
| fsigmoide=1/(1+np.exp(elem) for elem in net) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me parece que la sigmoide eleva a la -x
| fsigmoide=1/(1+np.exp(elem) for elem in net) | ||
| return fsigmoide | ||
| def tanhip(net): | ||
| ftanhip=((np.exp(elem)-np.exp(-elem))/(np.exp(elem)+np.exp(-elem)) for elem in net ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Estas devolviendo un vector? Tal vez la sintaxis debería ser
ftanhip = [(np.exp(elem)-np.exp(-elem))/(np.exp(elem)+np.exp(-elem)) for elem in net]
| @@ -0,0 +1,57 @@ | |||
| class perceptron: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Las clases por convención se declaran con Mayúsculas, esto es Perceptron
| @@ -0,0 +1,57 @@ | |||
| class perceptron: | |||
|
|
|||
| def insert_matriz(self): | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Recomendaría que no cargue elemento a elemento los datos. Esto esta bien para ejemplos tipo toy como AND y OR y XOR, pero si tengo cientos de datos? Preferible que cargue la matriz de datos X desde afuera y mas bien extraiga de la misma las filas y columnas para disponer del numero de muestras y del numero de componentes
| return learn_rate=0.1 | ||
| def learn2(self): | ||
| return learn_rate=0.01 | ||
| def learn3(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seguramente en esta parte Pycharm te pinta un warning, ya que este método al no utilizar self pasa a ser una funcion cualquiera y no es necesario que este dentro de la clase. Revisar
Este programa permite escoger el numero de elementos aleatorios que queremos del array y compara los tiempos de ordenamiento tanto del bubble sort como de insertion sort...
Buen dia!