axel9719 / pai-2020-tarea-08

Repo base para la tarea 08
0 stars 1 forks source link

Trabajo #1

Open axel9719 opened 3 years ago

axel9719 commented 3 years ago

Claudia este es el fork en el que te agregue, el otro que hiciste creo que no puedo hacer mi propia rama ni subir cosas.

Puse las funciones para el primer inciso en la carpeta con mi nombre y tengo un .c aparte con un menú donde estaba probando que funcionara como debería, igual ya tengo las complejidades del primero en un pdf, es parecido al del árbol binario pero base 3 pero creo que lo agregare como comentario en el codigo.

ClaudiaAD commented 3 years ago

Axel, OK. Yo no sé nada de git hub y por eso no había encontrado este mensaje. Por alguna razón me llegan notificaciones del fork que yo hice pero no de este. Como ya hiciste la primera parte, la revisaré y trataré mañana de tener lista la segunda parte, la de la mediana. La subo en cuanto la tenga adelantada para que puedas revisarla.

ClaudiaAD commented 3 years ago

Axel estuve mirando el código. Tengo un par de comentarios. Uno es justamente que no tiene ningún comentario y deberíamos ponerle. Yo los puedo poner, pero como lo escribiste tú, quizás sea más simple que tú mismo lo comentes. Otra cosa es que no me parece bien que cuando llamas a getmax usemos la palabra Salió, porque parece que el nodo se removió. Entonces podemos poner algo del estilo: el máximo es .... Además, vale especificar en algún lado que nuestro montículo es todo de 0's. Lo ideal yo creo que sería poner algún tipo de aviso o usar un contador para decir que el montículo está vacío y que no hay nodos que remover ni máximo que obtener, porque ahora el código devuelve siempre un valor máximo y remueve siempre aunque esté vacío en teoría. Me dices porfa si tú lo cambias o yo, o si opinas distinto. Por lo demás creo que está bien. Escribiré usando tus funciones lo de la mediana, porque también vi que cambiaste los nombres de algunas. Lo subo tan pronto lo tenga listo.

ClaudiaAD commented 3 years ago

También creo que nos conviene agregar una función que imprima el arreglo. Yo la puedo agregar. Así se puede ver bien que esté funcionando todo.

axel9719 commented 3 years ago

Hola Claudia si yo también e estado teniendo problemas con Git, nunca lo había ocupado. En la carpeta que tiene mi nombre tengo un archivo llamado heap.c , acabo de poner comentarios para las funciones, el de nombre test1.c no tiene comentarios y solo lo había estado usando para probar, hasta donde vi quieren que entreguemos la verificación usando el specs.c

Okey agrega la función y checo lo de la mediana en cuanto lo subas.

ClaudiaAD commented 3 years ago

Ok. Ya encontré el heap.c Lo de la mediana lo subo pronto.

ClaudiaAD commented 3 years ago

Agregué en heap.h la función print_Heap que imprime el montículo. También en test1.c la agregué, por si quieres probar que funciones. Además puse dos números más, el 4 para saber cuantos datos tiene el montículo, y el 5 para imprimirlo.

ClaudiaAD commented 3 years ago

Axel, subí lo que tengo hecho de la mediana. En la carpeta Claudia, en median.c está lo de la mediana. Puse una función extra que imprime los dos montículos usando la de print_Heap que ya había puesto.

Subí también un nuevo archivo heap.c. En ese, lo que encuentras distinto del tuyo, es que tuve que modificar los prototipos y particularmente las funciones de heapify para trabajar también con montículos mínimos que hacen falta para la mediana. En las pruebas, por si no lo dejé claro, si poner 'm' en tipo, toma un montículo min y si pones 'M' toma un montículo max.

Además, en la estructura, agregué un char type, que guarda el tipo para saber si el montículo es min o max.

En probando, hice algo parecido a lo que tenías en test1.c para probar que lo de la mediana funcione. Yo probé con varios grupos de números y me parece que sirve bien.

Me falta poner lo de la complejidad y la demostración de que calcula la mediana que lo incluiré en el reporte.

Creo que cada uno debería hacer el reporte de la parte que hizo. Es decir, tú de lo primero y yo de lo segundo.

Y falta lo del specs

axel9719 commented 3 years ago

Puse todo en una carpeta con el nombre de los dos

modifique la funcion print_Heap para que los imprima entre barras . El heap.c se quedo el que modificaste. Revise median.c y me parecio bien. Ya quedo el specs.c El archivo test1.c es el del inciso 1 El archivo test2.c es el de la mediana, aparte del specs.s estuve checando aqui que todo funcionara con varios ejemplos y estuvo bien.

axel9719 commented 3 years ago

Lo del reporte de la primera parte, pues como solo son las complejidades y dijeron que se podia poner como comentarios arriba de las funciones no se si lo dejamos asi? En esa carpeta borre el main.c y el makefile

ClaudiaAD commented 3 years ago

Pues si. Así lo dejamos. Yo escribo la segundo parte mañana en la mañana y lo entregamos.

ClaudiaAD commented 3 years ago

Axel, subí un archivo pdf con el reporte porque no logré escribir en esto de git. Subí también el .tex por si quieres agregar, quitar o mejorar algo. Por favor revisa que esté bien. Cuando lo apruebes ya lo podemos submitir al moodle y acabamos. Espero.

axel9719 commented 3 years ago

Okey ya lo revise, me parece bien, entonces si quieres ya lo subimos

ClaudiaAD commented 3 years ago

Perfecto. Sabes cómo??

axel9719 commented 3 years ago

No recuerdo si querían que lo subiéramos al moodle, tendríamos que descargar la carpeta y subirla, o con que este aquí el trabajo esta bien

ClaudiaAD commented 3 years ago

Bueno, intentemos lo de subir la carpeta. Supongo que aunque fue en equipo, habrá que subirla cada uno a su submission de la tarea. Cualquier cosa nos seguimos escribiendo.

ClaudiaAD commented 3 years ago

Ya lo descargué y subí a moodle sin problema.