search

Epi25 / epi25-edc

CRF design for Epi25 collaborative
3 stars 3 forks source link

GGE final check #65

Closed BrigidRegan closed 8 years ago

BrigidRegan commented 8 years ago

I have done a final overall check of the GGE and EE forms and have some finishing touches requests!

  1. “Age at first seizure” – change to “Age at first seizure (excluding classical febrile seizures)”
  2. Edit the “Date of last data collection” so that year can be unknown too – not sure how this could be coded? Or if it possible? Sorry if we have discussed this before.
  3. Can the age of onset correction also have a way to enter unknown? Perhaps can -9 be unknown so that you can override the computed onset age? A practical example is a boy who presents with a GTCS seizure at 6 years and there is a history of absences for a few years before then but the exact age of onset of the absences is unknown. The data would be entered as GTCS onset of 6 and the absences would be left blank for unknown. The computed onset would be 6 which is actually not true and should really be unknown.
  4. The Neuroimaging comments box needs to also appear when ‘Abnormal’ is selected in ‘Neuroimaging findings’
BrigidRegan commented 8 years ago

Sam also asked if the maternal and paternal DNA fields can be removed from the GGE form? Thanks

rolandkrause commented 8 years ago

OK, we might need to submit individual issues now.

  1. Done
  2. Leave the field empty. It's not very clean but we gain no information if people will not enter. It's a required field so people will get a warning if it's not filled in any case.
  3. I have changed all age of onset fields to include "-9". In the above example, -9/unknown should be entered as the start of the absence seizures!, not just any seizure, so we don't loose the seizure type. But what about onset of other seizure types? We might need to discuss this further and think of other examples, I'm afraid.
  4. Done. Bonus request from Sam: Done.
BrigidRegan commented 8 years ago

All looks good. For the EE form, “Age at first seizure” – change to “Age at first seizure (excluding classical febrile seizures)”

Thanks

rolandkrause commented 8 years ago

Shouldn’t that be “Age in months at first seizure (excluding classical febrile seizures)”

On April 1, 2016 at 16:31:47, BrigidRegan (notifications@github.com) wrote:

All looks good. For the EE form, “Age at first seizure” – change to “Age at first seizure (excluding classical febrile seizures)”

Thanks

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub

BrigidRegan commented 8 years ago

Yes!

From: Roland Krause notifications@github.com<mailto:notifications@github.com> Reply-To: rolandkrause/epi25 reply@reply.github.com<mailto:reply@reply.github.com> Date: Tuesday, 5 April 2016 at 3:48 AM To: rolandkrause/epi25 epi25@noreply.github.com<mailto:epi25@noreply.github.com> Cc: Brigid bregan@unimelb.edu.au<mailto:bregan@unimelb.edu.au> Subject: Re: [rolandkrause/epi25] GGE final check (#65)

Shouldn’t that be “Age in months at first seizure (excluding classical febrile seizures)”

On April 1, 2016 at 16:31:47, BrigidRegan (notifications@github.commailto:notifications@github.com) wrote:

All looks good. For the EE form, “Age at first seizure” – change to “Age at first seizure (excluding classical febrile seizures)”

Thanks

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/rolandkrause/epi25/issues/65#issuecomment-205706426