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

check EOS in crops.obic #134

Open
gerardhros opened this issue Jun 3, 2021 · 4 comments
Open

check EOS in crops.obic #134

gerardhros opened this issue Jun 3, 2021 · 4 comments
Labels
duplicate This issue or pull request already exists

Comments

@gerardhros
Copy link
Contributor

is it correct that EOS values can be NA in crops.obic
that results in a EOS balance of NA
and indirectly affects management score

@KeesDoolNMI
Copy link
Contributor

Additional thing to check: crops with 'productie' in their name have extremely high vales for eos. Perhaps some kind of loading error?

@BrentWHR
Copy link
Contributor

When I updated the table to include more crops I attempted to fill in values for crops based on similarity with already existing crops. Some of the crops in the current table are really rare, I wasn't sure (and still aren't) what to base EOS values on (such as madder, teff, buckwheat, poppies, heather, nature elements....)

@BrentWHR
Copy link
Contributor

@KeesDoolNMI did you mean the 13 crops mentioned in #162 ? There are more crops with 'productie in the name and these have regular EOS values ranging from 0-1790.

@KeesDoolNMI
Copy link
Contributor

@KeesDoolNMI did you mean the 13 crops mentioned in #162 ? There are more crops with 'productie in the name and these have regular EOS values ranging from 0-1790.

That range should be correct. So I suppose it should be the crops mentioned in that issues, yes

@BrentWHR BrentWHR added the duplicate This issue or pull request already exists label Apr 22, 2022
@BrentWHR BrentWHR mentioned this issue Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants