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

errors in Spanish WG and WS output #436

Closed
vmarchman opened this issue Apr 2, 2023 · 11 comments
Closed

errors in Spanish WG and WS output #436

vmarchman opened this issue Apr 2, 2023 · 11 comments

Comments

@vmarchman
Copy link
Contributor

Hi @HenryMehta It seems like the scoring is not working quite right for the Spanish long forms. Some things are not scoring and there are extra columns left over from the English.

Spanish WG: The following categories don't seem to be summing from the responses, so 0s are being populated when there are responses that should be counted.

Jugar a Ser Adulto
Imitacion de Otros Tipos de Actividades de Adultos

And, Late Gestures Summary score should be the sum of the following 3 sub-scores:
Acciones con Objetos
Jugar a Ser Adulto
Imitacion de Otros Tipos de Actividades de Adultos

The following columns should be deleted from the output for Spanish WG:
First Signs
Phrases
Imitation
Labeling
Words Understood
Words Produced
First Gestures
Games Gestures
Adult Gestures
Parent Gestures
Early Gestures
Later Gestures
Total Gestures

Spanish WS: The following columns don't appear to be summing from the responses:

Scombine - Maybe this is really Combinar palabras?
Formas de Verbos: Presente
Formas de Verbos: Sucedieron
Formas de Verbos: Imperativos

The following columns should be deleted from the output:
Total Produced
How Children Use Words
Word Forms
Combining
Complexity

And "exemplo" ==> "ejemplo"

@HenryMehta
Copy link
Collaborator

@vmarchman I have removed (I think) the unwanted columns, in both development and production.

Moving from exemplo to ejemplo is not straightforward and I need to think about the approach. Since we have to recalc all scores anyway when this goes lives, that might be when it is done, but I would still need to prove it works beforehand

I'm looking at the scoring now

@HenryMehta
Copy link
Collaborator

@vmarchman Could you tell me which Spanish WG you will be testing this on. I want to update the numbers on it and it takes so long to run I don't want to do more than required

Same for WS, but working on WG at the moment

@HenryMehta
Copy link
Collaborator

@vmarchman I have removed the incorrect titles and rerun the summaries. Could you confirm if the totals are still incorrect please

@HenryMehta
Copy link
Collaborator

(3) For Spanish WG, the gestures sections are still not right. It is not reading these columns, either in the csv or in the clinical report: Jugar a Ser Adulto Imitacion de Otros Tipos de Actividades de Adultos

I am really struggling to find the solution here mostly because I do not understand. I am looking at the data in Spanish WG looking for fields that relate to Jugar a Ser Adulto Imitacion de Otros Tipos de Actividades de Adultos
and I'm lost.
[Spanish_WG].csv

So let me explain how the scoring works and you might be able to guide me to the cause. We use the scoring file below to define scoring. Each score is defined by a title, category, measure and order. The title is self explanatory. The category relates to any row within Spanish_WG.csv file above in the specified category or categories. The measure is what is needed in order for the score to increase (usually yes), and the order is the order they appear in output.

Spanish_WG_scoring.json.zip

From a scoring perspective I believe we're looking at the score defined as follows:
```
{
"title" : "Imitacion de Otros Tipos de Actividades de Adultos",
"category" : "gestures_parents",
"measure" : "yes",
"order" : 12
},

This will refer to these items in the csv

item_511,barrtrap,gestures,gestures_adult,no; yes,Barre o trapea.,,,,gestures_adult
item_512,llavpuer,gestures,gestures_adult,no; yes,Trata de meter la llave en la puerta.,,,,gestures_adult
item_513,pegamart,gestures,gestures_adult,no; yes,Pega con un martillo.,,,,gestures_adult
item_514,persigna,gestures,gestures_adult,no; yes,Reza y/o se persigna.,,,,gestures_adult
item_515,escrbmaq,gestures,gestures_adult,no; yes,Trata de escribir a máquina.,,,,gestures_adult
item_516,juegleye,gestures,gestures_adult,no; yes,Juega a que está leyendo.,,,,gestures_adult
item_517,fumacig,gestures,gestures_adult,no; yes,Fuma un cigarro.,,,,gestures_adult
item_518,aguaplnt,gestures,gestures_adult,no; yes,Le echa agua a las plantas.,,,,gestures_adult
item_519,tocainst,gestures,gestures_adult,no; yes,"Trata de tocar un instrumento musical (guitarra, tambor, etc.).",,,,gestures_adult
item_520,manecoch,gestures,gestures_adult,no; yes,Juega a manejar el coche.,,,,gestures_adult
item_521,lavaplat,gestures,gestures_adult,no; yes,Lava los platos.,,,,gestures_adult
item_522,sacude,gestures,gestures_adult,no; yes,Sacude.,,,,gestures_adult
item_523,escrplum,gestures,gestures_adult,no; yes,Trata de escribir con un lápiz o una pluma.,,,,gestures_adult
item_524,hacehoyo,gestures,gestures_adult,no; yes,Trata de hacer un hoyo.,,,,gestures_adult
item_525,ponelent,gestures,gestures_adult,no; yes,Se pone unos lentes.,,,,gestures_adult

Can you tell me what is missing?

@vmarchman
Copy link
Contributor Author

vmarchman commented Apr 14, 2023 via email

@HenryMehta
Copy link
Collaborator

OMG! I'm so sorry. I just couldn't see it. I've made a fix and I'm deploying to dev now. But I will need to rerun the scoring for it to update. I'll set them to update later today so you can test in your evening if you fell so inclined

@vmarchman
Copy link
Contributor Author

vmarchman commented Apr 14, 2023 via email

@vmarchman
Copy link
Contributor Author

vmarchman commented Apr 17, 2023 via email

@HenryMehta
Copy link
Collaborator

@vmarchman We have to use the one that is in the benchmark json file. I can take it out but that will mean it needs taking out from everywhere it currently shows. Please confirm

@HenryMehta
Copy link
Collaborator

@vmarchman I am going to delete all the Spanish WG scoring data in dev. I am then going to rerun the scoring for your test study. I hope this will make clear where the issue is. At the moment there is too much data for me to look at

HenryMehta added a commit that referenced this issue Apr 18, 2023
HenryMehta added a commit that referenced this issue Apr 18, 2023
HenryMehta added a commit that referenced this issue Apr 18, 2023
@HenryMehta
Copy link
Collaborator

@vmarchman ok, I think I've found why we were getting Gestos Tarios twice (once with a stress) and why we weren't getting unisex results and I've corrected in dev. I still have no figures for Comprension temprana Percentile-sex, but I also don't have any benchmark data for this which is why it is giving an error

HenryMehta added a commit that referenced this issue May 12, 2023
* #358 Update fields and locale values

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #398 and #387  Short form and long form benchmark changes

* #398 and #387  Short form and long form benchmark changes

* #400 Remove unwanted options from interface

* #397 Only Adjusted Benchmark button shows those results

* #397 correct spelling for matches

* #362 Adding GA Script to base files

* #313 Correcting Dutch translation

* #308 Adding additional plots

* #308 Adding additional plots

* #397 Add space in percentile-both so it matches

* Amend study count

* Amend study count

* Amend study count

* Amend study count

* #387 English short form category for combines changed

* #387 English short form category for combines changed

* Merge masters

* Brookes codes

* Bump cairosvg from 2.5.1 to 2.7.0 in /webcdi

Bumps [cairosvg](https://github.com/Kozea/CairoSVG) from 2.5.1 to 2.7.0.
- [Release notes](https://github.com/Kozea/CairoSVG/releases)
- [Changelog](https://github.com/Kozea/CairoSVG/blob/master/NEWS.rst)
- [Commits](Kozea/CairoSVG@2.5.1...2.7.0)

---
updated-dependencies:
- dependency-name: cairosvg
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Brookes add intrument family functionality

* Brookes

* Brookes

* Adding codes

* Adding codes

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Researcher reminders

* Researcher reminders

* Researcher login reminders

* Researcher login reminders

* Amend release process

* Update static

* Update static

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Opt out shared if non chargeable form

* #426 Opt out shared if non chargeable form

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #424 Debug into PDF outputs

* #424 Debug into PDF outputs

* #424 English WG

* #424 Spanish WG

* #436 SummaryDataAdmin to delete records

* #436 SummaryDataAdmin to delete records

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* .env changes

* #424 Select Clinical Forms to Download

* fix

* #438 Spanish CAT Translations

* #424 remove clinical reports from general data download

* #424 Correcting Spanish scores

* #424 Correcting Spanish scores

* #424 update Spanish WG benchmarks

* #437 Adding footer text to output

* #424 Spanish Norms

* #437 Display footer on easch page of output

* #433 Amending Splash page

* #433 Amending Splash page

* #433 Amending Splash page

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #426 Trying to sort Gestos Tarios

* #426 Trying to sort Gestos Tarios

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* Adjusted data downloads

* Adjusted data downloads

* Adjusted data downloads

* Error message if no clinical template

* Error message if no clinical template

* Remove footer from Spanish Clinical forms

* Remove Comprension temprana percentil

* Footer only on English WS and WG

* Footer only on English WS and WG

* Footer only on English WS and WG

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* Crontab for scoring

* Crontab for scoring

* Moving some languages

* Moving some languages

* Moving some languages

* Moving some languages

* Cleanup

* Cleanup

* Cleanup

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* #441 Brookes Changes

* #441 Brookes Changes

* #441 Brookes Changes

* #441 text amends

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Hrlp file

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
HenryMehta added a commit that referenced this issue May 16, 2023
* #358 Update fields and locale values

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #398 and #387  Short form and long form benchmark changes

* #398 and #387  Short form and long form benchmark changes

* #400 Remove unwanted options from interface

* #397 Only Adjusted Benchmark button shows those results

* #397 correct spelling for matches

* #362 Adding GA Script to base files

* #313 Correcting Dutch translation

* #308 Adding additional plots

* #308 Adding additional plots

* #397 Add space in percentile-both so it matches

* Amend study count

* Amend study count

* Amend study count

* Amend study count

* #387 English short form category for combines changed

* #387 English short form category for combines changed

* Merge masters

* Brookes codes

* Bump cairosvg from 2.5.1 to 2.7.0 in /webcdi

Bumps [cairosvg](https://github.com/Kozea/CairoSVG) from 2.5.1 to 2.7.0.
- [Release notes](https://github.com/Kozea/CairoSVG/releases)
- [Changelog](https://github.com/Kozea/CairoSVG/blob/master/NEWS.rst)
- [Commits](Kozea/CairoSVG@2.5.1...2.7.0)

---
updated-dependencies:
- dependency-name: cairosvg
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Brookes add intrument family functionality

* Brookes

* Brookes

* Adding codes

* Adding codes

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Researcher reminders

* Researcher reminders

* Researcher login reminders

* Researcher login reminders

* Amend release process

* Update static

* Update static

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Opt out shared if non chargeable form

* #426 Opt out shared if non chargeable form

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #424 Debug into PDF outputs

* #424 Debug into PDF outputs

* #424 English WG

* #424 Spanish WG

* #436 SummaryDataAdmin to delete records

* #436 SummaryDataAdmin to delete records

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* .env changes

* #424 Select Clinical Forms to Download

* fix

* #438 Spanish CAT Translations

* #424 remove clinical reports from general data download

* #424 Correcting Spanish scores

* #424 Correcting Spanish scores

* #424 update Spanish WG benchmarks

* #437 Adding footer text to output

* #424 Spanish Norms

* #437 Display footer on easch page of output

* #433 Amending Splash page

* #433 Amending Splash page

* #433 Amending Splash page

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #426 Trying to sort Gestos Tarios

* #426 Trying to sort Gestos Tarios

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* Adjusted data downloads

* Adjusted data downloads

* Adjusted data downloads

* Error message if no clinical template

* Error message if no clinical template

* Remove footer from Spanish Clinical forms

* Remove Comprension temprana percentil

* Footer only on English WS and WG

* Footer only on English WS and WG

* Footer only on English WS and WG

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* Crontab for scoring

* Crontab for scoring

* Moving some languages

* Moving some languages

* Moving some languages

* Moving some languages

* Cleanup

* Cleanup

* Cleanup

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* #441 Brookes Changes

* #441 Brookes Changes

* #441 Brookes Changes

* #441 text amends

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Hrlp file

* Adding Email extract to Admin

* Adding Email extract to Admin

* Amend Brooes link

* update Dajngo

* update Dajngo

* update Dajngo

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
HenryMehta added a commit that referenced this issue May 16, 2023
* #358 Update fields and locale values

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #397 Add Adjusted age

* #398 and #387  Short form and long form benchmark changes

* #398 and #387  Short form and long form benchmark changes

* #400 Remove unwanted options from interface

* #397 Only Adjusted Benchmark button shows those results

* #397 correct spelling for matches

* #362 Adding GA Script to base files

* #313 Correcting Dutch translation

* #308 Adding additional plots

* #308 Adding additional plots

* #397 Add space in percentile-both so it matches

* Amend study count

* Amend study count

* Amend study count

* Amend study count

* #387 English short form category for combines changed

* #387 English short form category for combines changed

* Merge masters

* Brookes codes

* Bump cairosvg from 2.5.1 to 2.7.0 in /webcdi

Bumps [cairosvg](https://github.com/Kozea/CairoSVG) from 2.5.1 to 2.7.0.
- [Release notes](https://github.com/Kozea/CairoSVG/releases)
- [Changelog](https://github.com/Kozea/CairoSVG/blob/master/NEWS.rst)
- [Commits](Kozea/CairoSVG@2.5.1...2.7.0)

---
updated-dependencies:
- dependency-name: cairosvg
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Brookes add intrument family functionality

* Brookes

* Brookes

* Adding codes

* Adding codes

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes - email researcher if parent use invalid code

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Brookes Monthly report crontab

* Researcher reminders

* Researcher reminders

* Researcher login reminders

* Researcher login reminders

* Amend release process

* Update static

* Update static

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Added expiry to BrookesCode

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* Renewal Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Study OptOut Functionality

* #426 Opt out shared if non chargeable form

* #426 Opt out shared if non chargeable form

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #427 Demographic Opt Out functionality

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #426, #427 Added born early to no demographc and various for individual PDFS

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 English WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #427 Spanish WS Individual PDF

* #424 Debug into PDF outputs

* #424 Debug into PDF outputs

* #424 English WG

* #424 Spanish WG

* #436 SummaryDataAdmin to delete records

* #436 SummaryDataAdmin to delete records

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* #436 only process completed forms for stats

* .env changes

* #424 Select Clinical Forms to Download

* fix

* #438 Spanish CAT Translations

* #424 remove clinical reports from general data download

* #424 Correcting Spanish scores

* #424 Correcting Spanish scores

* #424 update Spanish WG benchmarks

* #437 Adding footer text to output

* #424 Spanish Norms

* #437 Display footer on easch page of output

* #433 Amending Splash page

* #433 Amending Splash page

* #433 Amending Splash page

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #439 Amending Navbard colours

* #426 Trying to sort Gestos Tarios

* #426 Trying to sort Gestos Tarios

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* #436 Update Spanish WG Benchmark

* Adjusted data downloads

* Adjusted data downloads

* Adjusted data downloads

* Error message if no clinical template

* Error message if no clinical template

* Remove footer from Spanish Clinical forms

* Remove Comprension temprana percentil

* Footer only on English WS and WG

* Footer only on English WS and WG

* Footer only on English WS and WG

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* CDI3 Benchmarks

* Crontab for scoring

* Crontab for scoring

* Moving some languages

* Moving some languages

* Moving some languages

* Moving some languages

* Cleanup

* Cleanup

* Cleanup

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* Amend Brookes Monthly Report

* #441 Brookes Changes

* #441 Brookes Changes

* #441 Brookes Changes

* #441 text amends

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Generate Expiry code for 31 May

* Hrlp file

* Adding Email extract to Admin

* Adding Email extract to Admin

* Amend Brooes link

* update Dajngo

* update Dajngo

* update Dajngo

* fix home page

* User Admin filter

* User Admin filter

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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

No branches or pull requests

2 participants