-
Notifications
You must be signed in to change notification settings - Fork 0
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
automated translation from coding to english #12
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only got the gist of it (which makes sense), just one comment for your consideration.
pheno_utils/pheno_loader.py
Outdated
@@ -64,6 +66,7 @@ class PhenoLoader: | |||
valid_stage (bool): Whether to ensure that all research stages in the data are valid. | |||
flexible_field_search (bool): Whether to allow regex field search. | |||
errors (str): Whether to raise an error or issue a warning if missing data is encountered. | |||
auto_translate (bool): Whether to automatically translate the data from coding to english. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider the following alternative:
instead of the word "translation", and a boolean value,
use something like preferred_language="coding"
(or a better similar name of your choice) that accepts ["hebrew", "english", "coding"].
this can be readily extended to other languages if needed.
also, perhaps in the config we can set the default value for preferred_language
(like other PhenoLoader args). maybe some customers will prefer an english default, and others a coding default.
but maybe you considered this and had a good reason to go for a boolean translation approach.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome
Automatically transforms decoded questionnaire answers
from code to english