knowit / folk-webapp

3 stars 0 forks source link

Kompetanse - radar diagram problem #397

Closed torjohannessen closed 1 year ago

torjohannessen commented 1 year ago

Problem med at radarens strek ikke går hele veien rundt , men stopper ved nullverdier. Radardiagram for kompetansekartlegging, både for enkeltpersoner og for akkumulert oversikt Kompetanse/kompetanseområder: Når det er null i verdi stopper den sirkulære streken, her bør alle null verdier antagelig settes til 0 i stedet. I øyeblikket synlig for "Softskills" og "Cybersikkerhet"

image

larsgus commented 1 year ago

Den underliggende årsaken her er at radar diagrammet kun er laget for å fungere med spørsmål som har type: "knowledgeMotivation". Alle spørsmål på softskills kategorien og et spørsmål på cybersikkerhet kategorien bruker type: "customScaleLabels". Disse har ikke dimensjonene knowledge og motivation, og gir dermed ikke mening å innkludere i diagrammet. Vi burde eksludere disse spørsmålene fra averageCompetenceAndMotivation rapporten.

larsgus commented 1 year ago

La til AND q.type = 'knowledgeMotivation' i spørringen for rapporten, ser ut til å fikse problemet.

SELECT competence, motivation, c.subCategory, c.category
FROM (SELECT AVG(a.knowledge) AS competence, q.topic as subCategory, c.text as category
      FROM kompetansekartlegging_answers AS a
               INNER JOIN kompetansekartlegging_questions AS q ON a.questionid = q.id
               INNER JOIN kompetansekartlegging_categories AS c ON q.categoryid = c.id
      WHERE c.text != 'Jobbrotasjon'
      GROUP BY q.topic, c.text) as c
         INNER JOIN (SELECT AVG(a.motivation) AS motivation, q.topic as subCategory, c.text as category
                     FROM kompetansekartlegging_answers AS a
                              INNER JOIN kompetansekartlegging_questions AS q ON a.questionid = q.id
                              INNER JOIN kompetansekartlegging_categories AS c ON q.categoryid = c.id
                     WHERE c.text != 'Jobbrotasjon'
                       AND q.type = 'knowledgeMotivation'
                     GROUP BY q.topic, c.text) as m ON c.subCategory = m.subCategory and c.category = m.category;

image

larsgus commented 1 year ago

Men vi må nok gjøre tilsvarende for andre rapporter