iinaki / tda-grupito

0 stars 0 forks source link

Corrección TP1 #1

Closed fdelu closed 5 months ago

fdelu commented 6 months ago

Buenas! Les dejo las correcciones del TP. La nota es un 8. Cualquier cosa que no entiendan o que me quieran preguntar, me pueden mandar un mail o un mensaje por Slack :)

4. 1. Demostración por Inversiones

La demostración en general está bien. Solo les dejo algunas observaciones:

5. Algoritmo

Está perfecto.

Como detalle totalmente irrelevante a la materia pero que si hace a las buenas prácticas de programación que por ahi les sirve (obviamente no les baja nada la nota esto), por lo general cuando una función modifica una estructura de datos in-place se suele no devolver la estructura modificada.

Fijense por ejemplo la misma documentación list.sort() de Python: "To remind users that it operates by side effect, it does not return the sorted sequence (use sorted() to explicitly request a new sorted list instance)". Cuando se devuelve la estructura modificada se sobreentiende que es una copia de la estructura original.

5.1. Complejidad

"Primero el metodo de ordenamiento de la lista de batallas, el cual asumimos que ordena en $O(n log n)$"

¿Por qué asumimos? Aclarenlo ustedes mismos como parte de su respuesta. Digan que su algoritmo utiliza un algoritmo de ordenamiento que ordena en $O(n \log n)$. Si se refieren a la implementación de Python pueden referir a la documentación.

6. Mediciones temporales

Los gráficos están bien. Algunos items para mejorar:

Otras observaciones