search

emartinm / lsql

Aplicación web para el aprendizaje de las bases de datos
MIT License
4 stars 4 forks source link

Memoria TFG #71

Closed Dashito14 closed 3 years ago

Dashito14 commented 3 years ago

Issue para aclarar dudas, correcciones y mejoras de la memoria.

emartinm commented 3 years ago

Menciones para todos, para asegurar que os lleguen los mensajes: @Dashito14 @iburgoa13 @Tamiflu9 @ivarui01 @jcorreas

iburgoa13 commented 3 years ago

Buenas, he tenido mañana libre y he estado viendo algo como se podría hacer mejorar en la memoria. 1- He conseguido poner los enumerados en el índice, es algo extraño , doc no te pilla 1. TITULO el 1 como numérico, hay que incorporarlo como al título para que lo pille. 2- He quitado lo de las herramientas y puesto aparte, como comentó Enrique eso al final habría que hacerlo dos veces si estuviera en introducción y me da que ahí habrá mucho jaleo y bastante ya en castellano. 3- He añadido más títulos, revisad y añadir y modificar lo que creáis chicos. 4- He cambiado el orden de conclusiones y aportaciones a la inversa como se habló. En parte poco más, va cogiendo forma el esqueleto, ya queda menos chicos!! Último empujoncito.

Dashito14 commented 3 years ago

Respecto a herramientas, ¿hay que poner Django, Oracle y todo esto o solo los editores y Github?

emartinm commented 3 years ago

Sobre Django tendréis que hablar porque es el framework básico del juez. Oracle habrá que mencionarlo por ser el SGBD que usa el juez, pero no creo que haya que extenderse mucho. Podéis generalizar el título de "Herramientas" a "Preliminares" para contar los aspectos tecnológicos que son necesarios para entender la memoria (sean herramientas, frameworks u otra cosa). Y ese mismo capítulo acomodaría bastante bien a las "plataformas antecesoras" que entiendo que son otros jueces aplicados en el aprendizaje.

Dejar los "inicios del juez" y la "historia del open source" como capítulos separados me parece bien por ahora, y si salen cortos se podrán fusionar en un capítulo "Estado inicial del juez y metodología de desarrollo"

jcorreas commented 3 years ago

Un comentario complementario al de Enrique: sobre Oracle no hace falta hacer una introducción a lo que es y las funcionalidades que ofrece, pues podemos suponer estos conocimientos en el lector (aparte de una frase inicial), pero creo que sí es relevante describir cómo se hace la comunicación con Oracle, como se resuelven las situaciones que se pueden producir (excepciones, errores, etc.), cómo se crea un entorno dentro de Oracle para realizar consultas sin interferencias con otros usuarios (hay que describir cómo se crea el usuario y el schema del usuario, si se destruye al final de cada consulta o al final de la sesión del usuario, si se genera un usuario con nombre único cada vez, así como los casos específicos de ejecución de procedimientos almacenados y de disparadores). También es importante describir en alguna parte de la memoria cómo se realiza el proceso de comparación de resultados y detección de errores de alumnos. Aunque esta parte ya esté presente en la versión de de la que partís para vuestro TFG, creo que es importante describirlo para poder hacer explícita la dimensión completa de la herramienta. Desde mi punto de vista, hacer un juez de SQL como este es bastante más complejo que hacer un juez de programación del tipo de DomJudge (sin quitarle mérito, que está muy bien hecho, pero la verificación de corrección se limita a comparación de textos).

Dashito14 commented 3 years ago

Hola @jcorreas. Hemos avanzado en la memoria en los puntos: Resumen, Introducción, Herramientas e Implementaciones. Por si puedes ir leyéndolo para el día de la reunión. Avanzaremos estos días algo más pero bueno, con lo que hay ahora yo creo que tenemos varios puntos a discutir en la reunión :D

Por cierto, es todo provisional, escrito rápido sin cuidar las formas, es para hablar un poco del contenido.

jcorreas commented 3 years ago

Bien, lo miro para la reunión de mañana.

Dashito14 commented 3 years ago

Como una de mis tareas es crear dos logros nuevos creo que queda un poco coja una sección de las implementaciones solo para esto. Había pensado en incluirla con la explicación de los logros inicial, y agregar una especie de

Tras finalizar otras implementaciones se crean dos logros nuevos para seguir motivando a los alumnos

obviamente con otra frase pero más o menos con ese significado.

emartinm commented 3 years ago

Me parece mejor aglutinarlo en un único apartado de la memoria, como dices, aunque el añadido final estuviera separado en el tiempo de la primera tarea y tuviera issue y PR propio.

Dashito14 commented 3 years ago

Si queréis podéis revisar el punto 7.13, el del sistema de logros. Falta una mejor redacción pero me gustaría que pudierais corregir el contenido ya que está un poco largo al haber metido dos tareas en una.

jcorreas commented 3 years ago

Creo que está bastante bien, en particular me parece bien separar los logros en 7.13.1 en dos grupos que se desarrollaron en momentos distintos. Respecto a 7.13.2, creo que se puede generalizar el texto del principio a todos los logros: donde se dice "...de la cual heredarían los 3 modelos de los logros..." se puede decir que heredan todos los modelos de los logros (es así, ¿no?). La expresión "desbloquear" habría que explicarla un poco, que ni a mí me queda clara (¿se ha bloqueado por algún motivo, o es que todavía no se ha conseguido el logro? En la captura de pantalla posterior, "logros pendientes" queda mucho más claro). Los dos últimos párrafos son los que se pueden mejorar más. Desde mi punto de vista no es conveniente escribir en primera persona (excepto en el apartado de contribución individual). Creo que es mejor introducir esta parte de otra forma. Por ejemplo: "La introducción de los dos últimos logros descritos en el apartado anterior planteó dificultades adicionales, en particular el logro "NumSolvedTypeAchievement", que considera la resolución de problemas de un mismo tipo, ya que los tipos de problemas pueden variar al introducir nuevos ejercicios..." (la redacción no es definitiva, seguramente la corregiré más adelante, pero da una idea de la forma de introducir esta parte).

iburgoa13 commented 3 years ago

Me cuelo también por aquí. @jcorreas @emartinm Acabados están mis puntos de Implementaciones y Contribuciones. Implementaciones: 7.1 7.2 7.3 7.4 7.5 7.9 Contribuciones: 8.1 Lo que sí obviar aún quizás el tema de verbos que tengo que ver alguna cosa más, pero más que nada por ver cómo está todo compuesto, si está correcto, etc. Me faltaría completar el punto 1 entero (Introducción) a lo largo de la semana lo haré y os lo haré llegar

Tamiflu9 commented 3 years ago

Podéis mirar los puntos: 7.7 7.12 7.19 (quitando el recuento de asserts y de comentarios que tengo que cambiarlo)

jcorreas commented 3 years ago

Hola a todos, no voy a poder revisar nada de la memoria del TFG esta semana. Miraré todo lo que hayáis añadido a la memoria el lunes, por si queréis añadir algo más durante estos días.

iburgoa13 commented 3 years ago

@jcorreas está "terminado" los puntos 1 y mis 7 (1,2,3,4,5,9) y 8. Me falta por completar los 7 con los usos ya que no he tenido mi portatil este finde y no he podido acceder a linux para hacer las capturas y demás. Si puedo esta noche adelanto cositas de ahí. Un saludo.

Tamiflu9 commented 3 years ago

@jcorreas al punto 4 de Plataformas antecesoras le puedes echar un ojo también? para ver más o menos que cosas modificar

jcorreas commented 3 years ago

Hola, @Tamiflu9 @iburgoa13 @Dashito14 @ivarui01. Estoy revisando los textos que vais introduciendo en la memoria. En muchos casos estoy haciendo correcciones directamente sobre el texto: aunque no es la mejor forma de ayudaros a mejorar la memoria, es la más rápida. En cualquier caso, no basta con aceptar mis cambios: tenéis que verificar el texto que he escrito yo y, si lo veis necesario, completarlo. Mi objetivo con estas correcciones no es tanto que quede mejor la memoria, sino que veáis otras posibles redacciones que mejoran la legibilidad del texto y que apliquéis estas mismas ideas en el resto de la memoria.

En otros casos os pongo comentarios para que seáis vosotros los que cambiéis la redacción del texto. Y en los casos en los que indico "no se entiende", no quiero decir que yo no lo entiendo, sino que un lector ajeno a este proyecto no lo va a entender o está mal redactado y para mejorarlo habría que volver a escribirlo. Con este comentario lo que quiero es que volváis a escribir el texto, no que me expliquéis lo que queréis decir.

iburgoa13 commented 3 years ago

Buenas, Tenemos una duda, había un apartado en el 7.X (ya no) que era el conflicto de Tamara y Dani (Ranking con Logros), hemos pensado ponerlo en el punto 6, ya que es un claro ejemplo de conflictos y podía tener mejor relación. El problema, que perdemos información ya que no podemos mencionar las tareas siguientes (descargar ranking con logros etc) pero en el 7 se queda una tarea muy coja. ¿Dónde encajaría mejor @jcorreas @emartinm? Muchas gracias.

jcorreas commented 3 years ago

Hola, @iburgoa13 @Tamiflu9 @Dashito14 (y @emartinm ). Este es un aspecto interesante de la metodología de desarrollo. Podéis añadir un (sub)apartado en el capítulo 6 (por ejemplo, "Resolución de conflictos de desarrollo" o algo así) en el que ponéis este ejemplo como un caso característico de este tipo de conflictos y describís brevemente las dos tareas (con sendas referencias al capítulo 7) para explicar el conflicto y su resolución. Y después en el capítulo 7 (preferiblemente en la última de las dos tareas, Logros, o en la última cronológicamente --creo recordar que era el ranking, pero no recuerdo) en el que detalláis en la sección de desarrollo la resolución de este conflicto, los cambios que ha supuesto, etc.

Algunos elementos como este deben estudiarse desde múltiples puntos de vista, por lo que deben aparecer en distintos capítulos (no se está repitiendo exactamente lo mismo, sino describiendo dos facetas del proceso de desarrollo).

ivarui01 commented 3 years ago

@jcorreas he corregido y ampliado el punto 5. Estado inicial de LearnSQL

Dashito14 commented 3 years ago

Buenas, necesitamos saber la versión de Linux que hay en la Facultad.

emartinm commented 3 years ago

Buenas, necesitamos saber la versión de Linux que hay en la Facultad.

Lo he contestado en un comentario en la memoria, pero por si acaso es Ubuntu 20.04

iburgoa13 commented 3 years ago

@jcorreas @emartinm @Tamiflu9 @Dashito14 Tenéis en la siguiente ruta (https://drive.google.com/drive/folders/1-XY8ZScqRUndfhg_GryK-2nxliT3zWG1) el TFG hasta la fecha de hoy. El miércoles comentamos, no es la definitiva, aún queda alguna corrección, faltan también la revisión del punto 5 de Iván y revisiones de algunos puntos sueltos más y realizar los puntos en Inglés cuando estén okey los demás. Si queréis hacer alguna anotación, podéis hacerla en el doc directamente. El miércoles comentamos, Un saludo.

jcorreas commented 3 years ago

Gracias Iker por la última versión. He revisado muy rápidamente el texto en google docs y lo habéis mejorado mucho. Aun así, tengo comentarios y sugerencias que incluir. En cualquier caso lo hablamos el miércoles en la reunión. Todavía no traduzcáis el texto de la introducción y las conclusiones, algunos comentarios son de esos dos capítulos.

Dashito14 commented 3 years ago

@jcorreas he aceptado muchos de los cambios que has sugerido en el punto 3, algunos los he retocado y he dejado dos comentarios de respuesta a tus cambios sugeridos. Te lo digo por aquí porque seguro que después de aceptar tantos cambios no habrás podido ver estas respuestas.

Tamiflu9 commented 3 years ago

Tengo dos dudas, los lenguajes de programación como : C, C++, Java,.. etc. como deben ir? Y las extensiones de los archivos: .sql, .xls, etc.. ? Es que hemos estado cambiando varias veces y no se al final que opción sería mejor. @jcorreas, @emartinm

jcorreas commented 3 years ago

Hola, creo que al final hemos dicho que los nombres propios (incluyendo lenguajes) deben ir en mayúsculas y sin cursiva. Las palabras en inglés (feedback) en cursiva. Si alguna frase está íntegramente en inglés, en cursiva y con comillas ("404 Not found").

Si no es correcto, decidlo para tener una referencia común.

Dashito14 commented 3 years ago

¿Y las extensiones que teníamos en Courier? Las dejamos en Courier o las ponemos en Arial. También hay correcciones de negrita pero eso no creo que quede bien, como nos digáis.

jcorreas commented 3 years ago

Bien, en ese caso yo prefiero que los textos literales de código o nombres de fichero aparezcan en Courier.

Si decidís lo contrario, me parecerá bien, pero lo importante es ser consistentes en todo el documento.

emartinm commented 3 years ago

La culpa es mía, que he cambiado algunos a Arial. Pero hacedlo tal y como os dijera Jesús en su momento.

Eso sí, en una frase como "...guardamos un fichero XLSX para poder..." lo único que debe estar en Courier es XLSX, no el espacio anterior ni el posterior porque entonces ocupa el doble y queda feote

Dashito14 commented 3 years ago

En cuanto al punto 8 estoy viendo las correcciones de Enrique. Las opiniones que no son contribuciones ¿no están bien para presentar la tarea? Así también se habla de motivación a los alumnos (por ejemplo) como contribución. Si hay que eliminar los puntos de opiniones se queda en una lista de tareas que hemos hecho. O no sé como hay que enfocar ese punto.

emartinm commented 3 years ago

Ya he terminado mi pasada y aunque he puesto algunos comentarios, realmente son todo cosas menores. Ahora revisaré los miles de mensajes que me han llegado por si hace falta que conteste. En resumen, las cosas que he detectado son:

emartinm commented 3 years ago

En cuanto al punto 8 estoy viendo las correcciones de Enrique. Las opiniones que no son contribuciones ¿no están bien para presentar la tarea? Así también se habla de motivación a los alumnos (por ejemplo) como contribución. Si hay que eliminar los puntos de opiniones se queda en una lista de tareas que hemos hecho. O no sé como hay que enfocar ese punto.

Nada, dejadlo como está, era solo un comentario.

emartinm commented 3 years ago

Hay otra cosa que no me termina de convencer, que es la frase final del número de asertos que ejecutan vuestros tests de unidad. Lo decís de varias maneras, con varios verbos diferentes, y a veces parece que sois vosotros los que durante la elaboración de los tests realizasteis X comprobaciones. Quizá podáis mencionar aserto y decir algo como:

En total, los tests de unidad generados para esta tarea comprueban X asertos.

o

Los tests de unidad generados para esta tarea comprueban un total de X asertos.
Dashito14 commented 3 years ago

Me parece bien cambiar esas frases, pero quizás mencionar los asertos es algo más confuso ya que no sé si aserto es algo que todo el mundo sabe a lo que se refiere, por eso hablamos de poner "comprobaciones"

emartinm commented 3 years ago

Yo también pensaba así, pero creo que en un contexto informático y concretamente en la sección de tests de unidad es un conocimiento que se podría suponer en el lector. En cualquier caso, que quede claro que son los tests de unidad los que realizan o están compuestos por esos asertos/comprobaciones/cosas.

Dashito14 commented 3 years ago

Para ir terminando apartados, hemos contestado a algunos de los comentarios. En el punto 3 he reescrito algunas cosas que he marcado en comentarios. También he añadido dos comentarios en 7.13.1 y 7.13.2 para cuando podáis verlos. Aviso por aquí, igual que las otras veces, porque es un caos los mensajes que nos llegan a todos y es normal que se pierdan.

iburgoa13 commented 3 years ago

Han subido el drive, una pregunta, venía una autorización para descargar y hacer todo lo que hicimos. He subido la que se hizo el otro día, ¿no habría problema, no? Estéticamente es algo diferente pero pide lo mismo. @jcorreas @emartinm

emartinm commented 3 years ago

Yo también la he visto y me parecía equivalente a la que firmamos salvo el formato, que es un poco más bonito. Si habéis comprobado con detalle que el texto es fundamentalmente el mismo, podemos dejar la que tenemos.

iburgoa13 commented 3 years ago

Así se quedará pues, dudo que pongan pegas. Muchas gracias.

Dashito14 commented 3 years ago

En cuanto a la presentación, tenemos 5 minutos cada uno. ¿Qué estructura es mejor? ¿Qué cada uno hable sus 5 minutos de seguido o vayamos alternando? Si lo hacemos seguido yo creo que rompería un poco la estructura de la presentación porque cada uno deberá hablar de alguno de los puntos (presentación, conclusión...) y además de los apartados hechos por él mismo. No sabemos bien como hacerlo, por si nos podéis guiar.

jcorreas commented 3 years ago

Para ir terminando apartados, hemos contestado a algunos de los comentarios. En el punto 3 he reescrito algunas cosas que he marcado en comentarios. También he añadido dos comentarios en 7.13.1 y 7.13.2 para cuando podáis verlos. Aviso por aquí, igual que las otras veces, porque es un caos los mensajes que nos llegan a todos y es normal que se pierdan.

@Dashito14 no he visto comentarios en el capítulo 3. Creo que he contestado los de 7.13, pero avisadme por aquí, que tengo bastante cacao con la cantidad de mensajes que se reciben y comentarios que aparecen en google docs.

También he terminado de revisar la traducción. Creo que está todo muy bien. Pero yo soy bastante neurótico con la versión final de documentos de esta dimensión y creo que os vendría muy bien hacer una lectura completa (después de descansar, para que sea de nuevas) para detectar erratas, posibles omisiones, etc.

Dashito14 commented 3 years ago

Si, vamos a hacer una reunión mañana seguramente para cerrar todo, leyendo todo y dejándolo listo.

emartinm commented 3 years ago

En cuanto a la presentación, tenemos 5 minutos cada uno. ¿Qué estructura es mejor? ¿Qué cada uno hable sus 5 minutos de seguido o vayamos alternando? Si lo hacemos seguido yo creo que rompería un poco la estructura de la presentación porque cada uno deberá hablar de alguno de los puntos (presentación, conclusión...) y además de los apartados hechos por él mismo. No sabemos bien como hacerlo, por si nos podéis guiar.

Yo creo que para la presentación mejor minimizar el cambio de presentadores. Podéis dividir la presentación en 3 partes y que cada uno cuente una (da igual si habláis de tareas que hay hecho otro, de hecho no podréis hablar de todas). Al final, podéis hacer una ronda rápida estilo Eurovisión para que cada uno diga sus contribuciones en inglés.

Para la presentación lo que tiene que quedar claro es:

Y creo que con eso y poco más ya habéis llenado los 20 minutos

Tamiflu9 commented 3 years ago

Hemos subido la versión final de la memoria al drive : https://drive.google.com/drive/u/0/folders/15LFzXMEF91TeIzkl_2dy9nBe0nr5sEiq

Por otro lado, la tarea de informar de los logros se va a quedar así. He dejado indicado en la memoria que quedaría pendiente modificar lo del mensaje agrupado en una frase.

emartinm commented 3 years ago

Por otro lado, la tarea de informar de los logros se va a quedar así.

Vaya, la verdad que no tenía ni idea, pensaba que lo ibas a terminar del todo.

He dejado indicado en la memoria que quedaría pendiente modificar lo del mensaje agrupado en una frase.

Entonces vamos a hacer una cosa, porque no me parece adecuado aceptar un PR cuando todavía le quedan comentarios abiertos (de la misma manera que no he aceptado ningún otro PR con comentarios abiertos). En lugar de incluir el enlace al PR incluye un enlace a la rama de tu proyecto y menciona que no se ha terminado por integrar por falta de tiempo para procesar todas las revisiones recibidas. Yo por mi parte lo que haré será cerrar el PR #88 sin integrar.

Tamiflu9 commented 3 years ago

Es que pensé solo tenía hasta el viernes o el sábado, pero he visto el correo que me mandaste el otro día. Estoy intentando arreglarlo y a ver si hoy consigo terminarlo. Si no, mañana como hay que subir la memoria hago lo que has dicho

emartinm commented 3 years ago

Yo te propongo una cosa: como la fecha de entrega de la memoria es el 15, trata de tener el código para el 15 por la mañana (opción A). Que nos da para tenerlo todo en ese momento, perfecto. Que no, déjalo en tu rama y enlaza eso en la memoria en lugar del PR (opción B). Eso siempre y cuando tengas tiempo disponible hasta el martes por la mañana. Si justo estos días necesitas todo el tiempo para estudiar, mi recomendación es ir directamente a la opción B.

Tamiflu9 commented 3 years ago

La opción A. La memoria ya estaría actualizada, por si mañana la queréis revisar.

emartinm commented 3 years ago

Gracias! Si, yo mañana quiero revisar el texto de esta última tarea de mostrar lo logros.