search

AGROFIMS / hagrofims

AGROFIMS main application repository
GNU General Public License v3.0
5 stars 1 forks source link

TraitAlias missing for ''Other crops'' #265

Closed celineaubert closed 4 years ago

celineaubert commented 4 years ago

hi @omarbenites I noticed that the TraitAlias, the TraitValidation and the TratDataType are not exported for ''other crop'' in the Excel file. Therefore, KDSmart script does not work and we can't export fieldbook to KDSmart. SOXX1591706549--FInt201807_Burkina Faso.xlsx

image

omarbenites commented 4 years ago

Now fixed, please check the next update.

celineaubert commented 4 years ago

Hi Omar,

now there are TraitAlias, but when 2 ''other'' crops are used in the experiment, KDSMart scritp does not work, as many TraitAlias are similar (see pic and [SOXX1591706549--FInt201807_Burkina Faso (3).xlsx]). KDSmart does not accept 2 similar TraitAlias.

Could you use the name of the crop instead of ''crop''(ex: niébé and sorghum) as you do for the TraitName (https://github.com/AGROFIMS/hagrofims/files/4786776/SOXX1591706549--FInt201807_Burkina.Faso.3.xlsx) ? This is the same issue for Crop Phenology measurement. they should all have different TraitAlias. image

omarbenites commented 4 years ago

the link has been updated fixing this error. Please, you may check again.

celineaubert commented 4 years ago

image Great! it's working. Thanks Omar.