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

EXNAME - include TRT_NAME if available #14

Open
chporter opened this issue Jan 29, 2013 · 4 comments
Open

EXNAME - include TRT_NAME if available #14

chporter opened this issue Jan 29, 2013 · 4 comments
Assignees

Comments

@chporter
Copy link
Member

For experiments with multiple treatments (e.g., HSC), all EXNAMEs are the same. It's very difficult to reference ACMO data.

Proposed solution: Add a column for TRT_NAME in the ACMO_meta.dat and ACMO.csv files. This column would be blank for survey-type data, but would be extremely useful for field experiments, such as HSC.

@ghost ghost assigned frostbytten Jan 29, 2013
@frostbytten
Copy link
Member

Would it make more sense to concatenate the TRT_NAME to the experiment if it exists, bypassing the need to have an unnecessary field (especially for models which don't handle treatments)

@chporter
Copy link
Member Author

Yes, that would work. But the concatenated string is no longer EXNAME, so it would need another label, like EX_TRT_NAME.

@frostbytten
Copy link
Member

Technically, I thought we were supposed to make it that way because of models that didn't support multiple treatments. EXNAME should be EX+TRT_NAME, only the DSSAT translator at this point should be splitting.

On Jan 29, 2013, at 4:17 PM, Cheryl Porter [email protected] wrote:

Yes, that would work. But the concatenated string is no longer EXNAME, so it would need another label, like EX_TRT_NAME.


Reply to this email directly or view it on GitHub.

@frostbytten
Copy link
Member

Actually, it's EXNAME__TRT_NO right now (@MengZhang is checking on that at the moment)

On Jan 29, 2013, at 4:17 PM, Cheryl Porter [email protected] wrote:

Yes, that would work. But the concatenated string is no longer EXNAME, so it would need another label, like EX_TRT_NAME.


Reply to this email directly or view it on GitHub.

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