Closed lilyclements closed 1 week ago
@lilyclements I am getting an "unused definition_id argument" when I try to complete the dialog as shown below
Is there anything wrong with the way I have chosen my def_ID? Here is the full error message
@jkmusyoka the error says “unused argument: definitions_id” this just means that you need to update your package. You’d want to update all three for lots of changes made too (unrelated to this, but just while you’re updating!) epicsawrap, epicsadata, and rpicsa
@lilyclements. This is now resolved, thanks. I "forced" the updating.
But I now get a different error when I run the dialog again - "unused tmin and tmax arguments". See the error message below. Could it be a variable name issue?
@jkmusyoka ah, the issue is that you have too recent of a version for the R-Instat version now! Damn. So, we don't want to have tmin/tmax in there anymore. You can do that by the R script for now, and I'll remove them in the dialog. Sorry about that.
I made a comment here about changes to make to the R-Instat dialog, meaning to write an issue but it slipped my mind. I'll do an issue now.
Great
Can I also add to your list by suggesting that "start season probabilities" be renamed to "crop definitions" on the main and the subsequent sub dialog. This change will make it easier for Met staff to associate the "crop def" data frame to this sub dialog. The start season probabilities could easily be associated to start of rains which is not the case here.
I hope you agree?
When this PR is merged, we get a new parameter into the "Export to Google Buckets" dialog.
This is very simple, and is just an additional label and ucrInput.
definitions_id
export_r_instat_to_buckets
(the base function, I think).