Skip to content
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

David ruiz dataverse explorers #1447 #1529

Conversation

Davichobits
Copy link
Contributor

He creado un nuevo proyecto llamado 02-dataver-explorers con el ánimo de resolver el issue #1447

En este nuevo proyecto se tiene modificado el readme donde ya no se pide a las estudiantes que utilicen los sets de data que se les entrega, si no que ahora se les solicita que creen un JSON con nueva data usando IA.

@unjust unjust changed the base branch from main to next August 16, 2023 21:12
@unjust unjust changed the base branch from next to main August 22, 2023 20:36
@unjust unjust changed the base branch from main to next August 22, 2023 20:37
Copy link
Contributor

@unjust unjust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Primera ronda de feedback

projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
@Davichobits Davichobits force-pushed the david-ruiz-dataverse-explorers-#1447 branch from f2427a8 to 8feb965 Compare August 23, 2023 22:06
@Davichobits Davichobits deleted the david-ruiz-dataverse-explorers-#1447 branch August 23, 2023 22:08
@Davichobits Davichobits restored the david-ruiz-dataverse-explorers-#1447 branch August 23, 2023 22:10
@unjust unjust reopened this Aug 23, 2023
@Davichobits Davichobits deleted the david-ruiz-dataverse-explorers-#1447 branch August 23, 2023 22:12
@Davichobits Davichobits restored the david-ruiz-dataverse-explorers-#1447 branch August 23, 2023 22:12
@unjust unjust reopened this Aug 23, 2023
@merunga
Copy link
Collaborator

merunga commented Aug 24, 2023

Gracias @Davichobits y @unjust .
Comentarios generales:

  • Hay q eliminar todas las datas y dar una sola data por defecto, no?
  • No veo en el README la explicación de q si no consigues tener tu data en un sprint, debes usar esa.
  • Y tampoco veo referencias de cómo testear la correctitud JSON
  • Creo q el JSON está demasiado largo
  • @unjust ese archivo EXTRA.md sigue teniendo sentido?

package.json Show resolved Hide resolved
projects/02-dataverse-explorers/EXTRA.md Outdated Show resolved Hide resolved
Comment on lines +1 to +15
# FAQ (preguntas frecuentes)

- [Diferencia entre array y objetos](https://youtu.be/mJJloQY7A8Y)
- [¿Cómo agrego una nueva propiedad a un objeto?](https://youtu.be/mJJloQY7A8Y?t=236)
- [¿Cómo puedo recorrer un objeto?](https://youtube.com/01RHn23Bn_0)
- [map, filter, sort y reduce también son métodos para objetos](https://youtu.be/bUl1R2lQvKo)
- [Diferencia entre expression y statements](https://youtu.be/wlukoWco2zk)
- [Diferencia entre createElement e innerHTML](https://www.javascripttutorial.net/javascript-dom/javascript-innerhtml-vs-createelement/)
- [¿Qué es el Scope?](https://youtu.be/s-7C09ymzK8)
- ¿Qué es git y por qué debo usarlo?
- GitHub Colaborativo
- ¿Qué es un Merge y existen alternativas? (Rebase)
- ¿Existe una "buena forma" de usar git?

[Para estas preguntas sobre Git recomendamos ver este playlist](https://www.youtube.com/watch?v=F1EoBbvhaqU&list=PLiAEe0-R7u8k9o3PbT3_QdyoBW_RX8rnV)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

También puedo revisar si esta archivo sigo siendo útil.

projects/02-dataverse-explorers/README.md Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Outdated Show resolved Hide resolved
projects/02-dataverse-explorers/test/json.spec.js Outdated Show resolved Hide resolved
@@ -10,7 +10,7 @@ describe('JSON', ()=>{
})

it('cada objeto debe tener las propiedades "name", "img", "statistics", y "description"', () => {

expect.assertions(96)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Porque es 96 si el longitud de data es 24 ?

@@ -21,11 +21,16 @@ describe('JSON', ()=>{
});

it('la propiedad "statistics" debe tener como valor un objeto con las propiedades "value" y "label"', () => {

expect.assertions(48)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

quiza ocn eso si puede tener una variable cantidad de estadisticas podemos hacer solo hasAssertions

});

it('La propiedad value del objeto statistics debe ser un número', ()=>{
jsonData.forEach(item => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tambien aqui expect.assertions

* El tiempo estimado que deberías dedicar a la generación de la data
es de máximo un sprint. Si ves que te va a tomar más tiempo, te
recomendamos utilizar la data de ejemplo que la vas a encontrar en
esta ruta: `./src/data/data.json`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -0,0 +1,36 @@
import jsonData from '../src/data/data.json'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pdemos mover eso dentro los tests/read-only cuando movemos los archivos (jueves)

Copy link
Collaborator

@ssinuco ssinuco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incluí varios cambios que habia enviado al proyecto Datalovers en marzo que me parecen utiles y estan este PR #1336 que se discutio al inicio pero no lo retomamos

Tambien me gustaria:

  • centralizar la informacion, siento que en un parte dice "Una vez que entiendas las necesidades de tus usuarios, escribe las Historias de Usuario que representen todo lo que el usuario necesita hacer/ver." y mas adelante se habla de cómo identificar las necesidades de los usuarios "Antes de empezar a escribir código, debes definir qué deberá hacer el producto en base al conocimiento que puedas obtener de tu usuario. Estas preguntas te pueden ayudar:". Siento que esta todo disperso.
  • eliminar la seccion "Contenido de referencia" porque suele ser ignorada por las estudiantes y mas bien distribuir esos links en el README

Comment on lines +37 to +57
En este proyecto utilizarás herramientas de
[inteligencia artificial](https://es.wikipedia.org/wiki/Inteligencia_artificial)
como [ChatGPT](https://openai.com/chatgpt), [ExplainDev](https://explain.dev/),
entre otras para generar set de datos en formato
[JSON](https://es.wikipedia.org/wiki/JSON).

El proposito de generar la data en esta manera es brindarte la oportunidad de
adentrarte en el empleo de herramientas impulsadas por la inteligencia
artificial, así como en tecnicas de
[Prompting](https://learnprompting.org/es/docs/intro).

Además **construirás una _página web_ para visualizar el
_conjunto (set) de datos_** generado anteriormente. Esta página web se
adecuará a lo que descubras que tu usuario necesita.

Como entregable final tendrás una página web que permita **visualizar la data,
filtrarla, ordenarla y hacer algún cálculo agregado**. Con cálculo agregado
nos referimos a distintos cálculos que puedes hacer con la data para mostrar
información aún más relevante para los usuarios (promedio, el valor máximo
o mínimo, etc).

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
En este proyecto utilizarás herramientas de
[inteligencia artificial](https://es.wikipedia.org/wiki/Inteligencia_artificial)
como [ChatGPT](https://openai.com/chatgpt), [ExplainDev](https://explain.dev/),
entre otras para generar set de datos en formato
[JSON](https://es.wikipedia.org/wiki/JSON).
El proposito de generar la data en esta manera es brindarte la oportunidad de
adentrarte en el empleo de herramientas impulsadas por la inteligencia
artificial, así como en tecnicas de
[Prompting](https://learnprompting.org/es/docs/intro).
Además **construirás una _página web_ para visualizar el
_conjunto (set) de datos_** generado anteriormente. Esta página web se
adecuará a lo que descubras que tu usuario necesita.
Como entregable final tendrás una página web que permita **visualizar la data,
filtrarla, ordenarla y hacer algún cálculo agregado**. Con cálculo agregado
nos referimos a distintos cálculos que puedes hacer con la data para mostrar
información aún más relevante para los usuarios (promedio, el valor máximo
o mínimo, etc).
En este proyecto *construirás una _página web_ para visualizar,
filtrar, ordenar y agregar un _conjunto (set) de datos_
en formato
[JSON](https://es.wikipedia.org/wiki/JSON)** generados con herramientas de
[inteligencia artificial](https://es.wikipedia.org/wiki/Inteligencia_artificial)
como [ChatGPT](https://openai.com/chatgpt) o [Bard](https://bard.google.com/).
El propósito de generar los datos con estas herramientas es aprender
técnicas de
[Prompting](https://learnprompting.org/es/docs/intro)
para interactuar con inteligencias artificiales.
Con agregar datos
nos referimos a distintos cálculos que puedes hacer con la data para mostrar
información aún más relevante para los usuarios (promedio, el valor máximo
o mínimo, etc).

projects/02-dataverse-explorers/README.md Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Show resolved Hide resolved
projects/02-dataverse-explorers/README.md Show resolved Hide resolved
Comment on lines +222 to +225
* En lugar de consumir la data estática brindada en este repositorio, puedes
consumir la data de forma dinámica, cargando un archivo JSON por medio de
`fetch`. La carpeta `src/data` contiene una versión `.js` y una `.json` de
de cada set datos.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto ya no aplica.

projects/02-dataverse-explorers/README.md Show resolved Hide resolved
### Primeros pasos

Súmate al canal de Slack
[#project-data-lovers](https://claseslaboratoria.slack.com/archives/C03MV35EP5M)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seria un nuevo canal

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hay que tenerlo en cuenta, este canal aún no existe.

Comment on lines +437 to +441
* [ ] Incluye _sketch_ de la solución (prototipo de baja fidelidad) en
`README.md`.
* [ ] Incluye _Diseño de la Interfaz de Usuario_ (prototipo de alta fidelidad)
en `README.md`.
* [ ] Incluye link a Zeplin en `README.md`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [ ] Incluye _sketch_ de la solución (prototipo de baja fidelidad) en
`README.md`.
* [ ] Incluye _Diseño de la Interfaz de Usuario_ (prototipo de alta fidelidad)
en `README.md`.
* [ ] Incluye link a Zeplin en `README.md`.
* [ ] Incluye prototipo de baja fidelidad de la solución en `README.md`.
* [ ] Incluye prototipo de alta fidelidad de la solución en `README.md`.

Comment on lines +444 to +446
* [ ] UI: Muestra lista y/o tabla con datos y/o indicadores.
* [ ] UI: Permite ordenar data por uno o más campos (asc y desc).
* [ ] UI: Permite filtrar data en base a una condición.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [ ] UI: Muestra lista y/o tabla con datos y/o indicadores.
* [ ] UI: Permite ordenar data por uno o más campos (asc y desc).
* [ ] UI: Permite filtrar data en base a una condición.
* [ ] Despliegue en Github Pages
* [ ] UI: Muestra la data en una interfaz: puede ser un card, una tabla, una lista,
* [ ] UI: Ordenar la data por al menos dos criterios.
* [ ] UI: Filtrar la data por al menos dos criterios.
* [ ] UI: Calcular una estadística de la data.

Copy link
Collaborator

@ssinuco ssinuco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incluí varios cambios que habia enviado al proyecto Datalovers en marzo que me parecen utiles y estan este PR #1336 que se discutio al inicio pero no lo retomamos

@unjust unjust closed this Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dataverse: Data-lovers agrega prompting a la creación del dataset y se convierte en Dataverse
4 participants