search

openpolitica / open-politica-backend

Backend del Proyecto Open Política construido en Node.js
Apache License 2.0
0 stars 0 forks source link

Create the algorithm to divide results by topic #125

Closed dmarquinah closed 3 years ago

dmarquinah commented 3 years ago

Se ha modificado la función getPolicyResults para separar la compatibilidad por tópico enviado.

Y enviando el response con el formato para cada partido: { "name": "...", "org_politica_id": ..., "org_politica_nombre": "...", "compatibility": { "education": "0.67", "health": "1.00" }, ... }

dmarquinah commented 3 years ago

@gianpietro1 @LuighiV si pueden probar y revisar que algo no se me haya pasado sería excelente, dejé comments pero igual pueden preguntarme, faltaría de igual manera agregar la lógica para lo de las respuestas múltiples.

gianpietro1 commented 3 years ago

@dmarquinah consideras necesario pedir el topic? es que ese mapeo de codPregunta a topic ya lo tenemos en DB, y sería más simple implementar el cálculo sin hacer el cambio en front?

gianpietro1 commented 3 years ago

@dmarquinah algo que sería bueno añadir sería el puntaje total también, considerando todos los topics, porque la idea es también mostrarlo

dmarquinah commented 3 years ago

@dmarquinah consideras necesario pedir el topic? es que ese mapeo de codPregunta a topic ya lo tenemos en DB, y sería más simple implementar el cálculo sin hacer el cambio en front?

Es una posibilidad, esta es la primera versión asi que se puede cambiar, yo lo uso actualmente para el conteo de preguntas por topic

dmarquinah commented 3 years ago

@dmarquinah algo que sería bueno añadir sería el puntaje total también, considerando todos los topics, porque la idea es también mostrarlo

ok, lo entendí y tienes razón, sería un campo más en el key 'compatibility' 👍

gianpietro1 commented 3 years ago

@dmarquinah consideras necesario pedir el topic? es que ese mapeo de codPregunta a topic ya lo tenemos en DB, y sería más simple implementar el cálculo sin hacer el cambio en front?

Es una posibilidad, esta es la primera versión asi que se puede cambiar, yo lo uso actualmente para el conteo de preguntas por topic

Creo que mejor sin pedir el topic por ahora, para poder probar sin cambios en el request?

dmarquinah commented 3 years ago

@LuighiV de hecho así está el ejemplo, con una pregunta repetida: https://github.com/openpolitica/open-politica-backend/pull/125/files#diff-7fbc47e3e759242905f778881d685ff8bcf1133e226ce9d30e6ffbf27393e7e5R553

LuighiV commented 3 years ago

@LuighiV de hecho así está el ejemplo, con una pregunta repetida: https://github.com/openpolitica/open-politica-backend/pull/125/files#diff-7fbc47e3e759242905f778881d685ff8bcf1133e226ce9d30e6ffbf27393e7e5R553

Sorry, pensé que decía sal2 :+1:

gianpietro1 commented 3 years ago

@dmarquinah mi único comentario es si ves posible y estás de acuerdo en reducir cambios en front, haciéndolo así:

      "compatibility": "0.70",
      "compatibility_per_topic": {
        "education": "0.67",
        "health": "0.75",
      }
dmarquinah commented 3 years ago

@dmarquinah mi único comentario es si ves posible y estás de acuerdo en reducir cambios en front, haciéndolo así:

      "compatibility": "0.70",
      "compatibility_per_topic": {
        "education": "0.67",
        "health": "0.75",
      }

Si! aún falta agregar ese cambio