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

Collecting data for plots that are segregating. #7

Open
laceysanderson opened this issue Apr 28, 2017 · 3 comments
Open

Collecting data for plots that are segregating. #7

laceysanderson opened this issue Apr 28, 2017 · 3 comments

Comments

@laceysanderson
Copy link
Member

Some of our plots are segregating for specific phenotypes. For example, some plots might be segregating for flower colour (e.g. white/purple) or days to flower (e.g. 42/59 days). In these cases, some data collectors will record both phenotypes they observed (as shown in brackets above). Kirstin feels the loader should handle this.

@reynoldtan
Copy link
Member

I don't see any problem with traits with text unit, but for numeric units such as count, days, centimeters and grams (we coded it to expect positive whole numbers, plus scale which is 1-5), yes the module will detect this as an error. A workaround might be to just set the unit to anything relevant (e.g. Flower Colour (colour), Days to Flower (seg-days) or (#ofdays) etc.) but the list of unit mentioned under numeric.

For existing AGILE trait, an option would be to record both values in separate rows and let the download page group the values, or in terms of code, we can tell the module to be a little tolerant if it detects "/" or "," (or other chars that connote an enumeration/segregation) when recording numeric values.

Please advice

@laceysanderson
Copy link
Member Author

I think we should make the module more tolerant.

Solution:

When checking if a numeric value is numeric:
If it fails then check if there is a "/" or "," character and that the remaining characters on either side of such a symbol are numeric.

@laceysanderson
Copy link
Member Author

I would prioritize #15 and #8 before working on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants