search

Epi25 / epi25-edc

CRF design for Epi25 collaborative
3 stars 4 forks source link

EE offset ages #54

Closed BrigidRegan closed 8 years ago

BrigidRegan commented 9 years ago

Can you please change the field notes for the EE seizure types to 'Age in years at last occurrence'? There have been a few problems with dealing with large numbers of months and Sam suggested this as a way around it!

rolandkrause commented 9 years ago

I implemented the change and updated the data quality rules accordingly. 

We should discuss however whether it wouldn’t be better to calculate the age given the input in month, so we would stay consistent within the form. 

Input could be given in years, months would be calculated for display and used internally as values to pass on.

On November 12, 2015 at 23:41:45, BrigidRegan (notifications@github.com) wrote:

Can you please change the field notes for the EE seizure types to 'Age in years at last occurrence'? There have been a few problems with dealing with large numbers of months and Sam suggested this as a way around it!

— Reply to this email directly or view it on GitHub.

BrigidRegan commented 8 years ago

That sounds like a good plan to me

BrigidRegan commented 8 years ago

I checked the idea above of the years being converted to months for display with Sam. He wasn't keen on this as it would result in false precision but is happy to be convinced if you think it is necessary.