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

Agregar data y AI prompting para dataverse #1560

Merged

Conversation

Davichobits
Copy link
Contributor

@Davichobits Davichobits commented Sep 6, 2023

#1447

Esta listo el PR de dataverse con tests de OA y prompting para poder recibir feedback.

Actualmente estamos dudando sobre la ubicación de estas dos secciones:

La parte de "Generar la data" dentro del apartado 5. Criterios de aceptación mínimos del proyecto ?
El archivo json.spec.js dentro de la carpeta test-read-only ?

Si tienen una mejor idea o algún feedback les agradecemos mucho!

@unjust unjust self-requested a review September 7, 2023 17:36
@unjust unjust requested a review from merunga September 7, 2023 17:36
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.

Algunas comentarios de @merunga que podemos hacer aca. Tenemos un poco tiempo hasta que necesitamos cerrar este PR y podemos avanzar con #1510.

@unjust
Copy link
Contributor

unjust commented Sep 13, 2023

Estamos cerca, creo los pendientes serian:

  • Actualizamos el readme y el json para usar un archivo js podemos nombrarlo dataset.js
  • verificar que el tests de OAs corre, incluyendo lo de data
  • Revisar que estamos consistente con las instrucciones de usar URLs de imagenes
  • Revisar el feedback de Sergio en su pull Using export not fetch Davichobits/curriculum#2 para ver si hay algo mas que deberia cambiar
    Falta algo?

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.

Algunas preguntas y comentarios

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.

consistencia con dataset.js y cambios en el test

En internet podrás encontrar varias páginas para generar imágenes con
IA, puedes aventurarte a buscar y quedarte con la que más te guste.
O puedes usar alguna de estas dosrecomendaciones: [GenCraft](https://gencraft.com/generate)
o [Hugginface](https://huggingface.co/spaces/laboratoria/dataverse-image-generation)
Copy link
Collaborator

Choose a reason for hiding this comment

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

esto no se si deberiamos ponerlo aqui todavia, no hay ningun tipo de control de consumo

Copy link
Contributor Author

@Davichobits Davichobits Sep 18, 2023

Choose a reason for hiding this comment

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

@merunga te refieres a no sugerirles que busquen en internet una herramienta generadora de imágenes si no que directamente usen la que proponemos?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Esta @Davichobits
https://www.bing.com/create
Pruebala a ver q te parece y la agregamos en lugar de la nuestra de hugging face, inclusive antes de GenCraft la pondria.


#### Diseño de la Interfaz de Usuario
#### Diseño de la Interfaz de Usuaria

##### Prototipo de baja fidelidad
Copy link
Collaborator

Choose a reason for hiding this comment

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

esta seccion sigue teniendo sentido al darle los mockups?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Los mockups que les proponemos son de referencia, a veces los toman pero mayormente proponen otros diseños.

@@ -501,14 +536,14 @@ Recuerda pedir feedback de tu prototipo a tus compañeras y/o coaches.

#### Testeos de usabilidad
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esto sucede en la realidad? Tiene sentido al darles los mockups?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yo diría que si, como mayormente ellas proponen su propio diseño, sería necesario el testeo de usabilidad.

Copy link
Contributor

Choose a reason for hiding this comment

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

Si! Algunas lo hacen.
Y no importa si usan los mockups, hay que ver si el producto tal cual hace sentido y los controles son usables.

Creo seria bueno dar mejor fuerza mostrar como hacerlo en los bootcamps.

En internet podrás encontrar varias páginas para generar imágenes con
IA, puedes aventurarte a buscar y quedarte con la que más te guste.
O puedes usar alguna de estas dosrecomendaciones: [GenCraft](https://gencraft.com/generate)
o [Hugginface](https://huggingface.co/spaces/laboratoria/dataverse-image-generation)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esta @Davichobits
https://www.bing.com/create
Pruebala a ver q te parece y la agregamos en lugar de la nuestra de hugging face, inclusive antes de GenCraft la pondria.

Comment on lines 89 to 91
console.error(`El objeto con el nombre: ${item.name} no cumple con la condición, tiene ${words.length} palabras`);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esto sientes q hace falta @Davichobits ? porque justamente el expect va a fallar

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A mi me sirvió mucho. Algunas de las descripciones que había generado no estaban cumpliendo con las 100 palabras como mínimo. Este console.error me ayudó para no estar revisando objeto por objeto a ver a cual le faltaba. (En el error que me daba el test no me arrojaba que línea es la producía el error.)

@unjust unjust merged commit 9aff0f9 into Laboratoria:feat/dataverse-part-1 Sep 20, 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
5 participants