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

Total sugar content #67

Open
rtbbase-helper opened this issue Jun 13, 2018 · 5 comments · May be fixed by #75
Open

Total sugar content #67

rtbbase-helper opened this issue Jun 13, 2018 · 5 comments · May be fixed by #75

Comments

@rtbbase-helper
Copy link

Submitter

Name Jolien Swanckaert
Program CIP Ghana

Request

Trait Name Total sugar content
Definition Total sugar content measured by NIRS.
Trait Class Quality
Measurement Method Measurement
Measurement Type Unit
Measurement Unit g/100g DW
@jswancka
Copy link

Total sugar content estimating 1-9 does exist in the ontology, but we want Total sugar content measured by NIRS in g/100g DW.

@jswancka
Copy link

Now I am confused. I think it should be percentage. Can you check with other sugar information, because these have no unit defined in the ontology (for example sucrose content does not specify the unit).

@dgemenet
Copy link

dgemenet commented Aug 1, 2018

Normally we use % for sugars from NIRs readings. So it should be % total sugar.

@bellerbrock
Copy link
Contributor

Hi Jolien and Dorcus, no problem, I can edit the total sugar trait so it has the same % unit as the other NIRS sugar traits. I assume there should be both a raw and cooked version of this trait as well?

@bellerbrock
Copy link
Contributor

Also, I see that the starch content trait clarifies that the percentage is percent dry weight. Are all the sugar traits expressed as percent dry weight as well?

@bellerbrock bellerbrock linked a pull request Oct 1, 2018 that will close this issue
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants