OpenSprinkler › Forums › OpenSprinkler Mobile and Web App › Error when saving Options
- This topic has 15 replies, 5 voices, and was last updated 9 years, 3 months ago by Ray.
-
AuthorPosts
-
September 10, 2015 at 2:12 pm #40179
ArndParticipantHi,
my name is Arnd and today I received my open sprinkler controller (after sitting in German customs for 2 weeks), which is a big improvement over my 10+ year old Gardena controller, so I am quite happy about it.
Everything seems so be working just fine except for two minor things:
When I go to the options dialog and I try to save the modified options whatever I do I get the message: “Please check input and try again” (or the German equivalent).
However, it seems that all changes are saved in spite of the error message. So this is not a big problem, but still a bit irritating.The second thing I noticed is that I cannot use a wunderground weather station near to me. When I enter the “Map-picker dialog” I am able to locate a station very close to my house and select it. The entry changes correctly to “Kirrlach, Baden-Württenberg”. I save it (get the error msg, but never mind) and when I re-enter the dialog, I get “Altlussheim, Baden-Württemberg”, maybe 10km away. Are there some limitations in wunderground which prevent a higher accuracy with the free service?
Any help appreciated!
Arnd.September 10, 2015 at 3:13 pm #40180
SamerKeymasterHi Arnd,
Regarding the error message you are seeing, which OpenSprinkler are you using and also which firmware?
The way the location works in the app is the GPS coordinates are saved (and not the weather station name). This means you can put a point anywhere you want, including your home address. Weather Underground will resolve the coordinates to the closest weather station (and if it matches the coordinates of the saved station there will be no ambiguity).
The app shows the location by asking Google to geolocate the coordinates. I use Google because the location names provided by Weather Underground are variable and seem user generated.
Thank you,
SamerSeptember 10, 2015 at 3:21 pm #40181
ArndParticipantHi Samer,
thanks for your fast reply!
Sorry I didn´t mention the details in my previous post, I am using the DC Version 2.3 with Firmware 2.1.5 (2).Thanks for the clarification concerning the weather location, that makes sense.
Really a great controller! I spend the last 30 mins looking at the android app watching the stations cycle throug the programs, thats really cool.
Arnd
September 10, 2015 at 3:24 pm #40182
SamerKeymasterI admit full testing with the latest app against the DC controller was not performed and I will take a look at this right away to identify the issue.
Thank you for letting us know!
September 10, 2015 at 3:25 pm #40183
ArndParticipantThank you for looking into it, Samer!
September 10, 2015 at 7:19 pm #40186
SamerKeymasterWe don’t see an issue with the DC controller so it might be an option setting you have causing the issue.
Do you mind exporting your configuration and emailing it to [email protected] (I want to discourage pasting configuration’s here since they contain API keys)? This will allow me to replicate your options and hopefully the error.
Thank you!
September 13, 2015 at 6:43 am #40195
ArndParticipantI have sent the config to [email protected]
I have noticed one more thing concerning the options which may or maybe not related to the problem stated in this thread: When I modify the Zimmerman parameters, click on save in the parameter dialog everything is fine. But when I re-enter the parameter dialog (just after having closed it after saving) the old 100/100/100 values are still present. Even when I export my config, set the values to something different than 100/100/100 by editing the config in an editor and then re-importing the config the parameter dialog will still show 100/100/100. As the water level% display seems to have changed after importing the modified parameters I suspect that the parameter dialog is just showing the wrong values (always 100/100/100). I wonder anyone else can confirm this behaviour running firmware 2.1.5.
Arnd
September 13, 2015 at 9:55 am #40197
DennisParticipantI too am seeing the “Please check input and try again” message. My settings are being saved despite it. Also, my “Weather Adjustment Options” dialog has always had that reset issue. I found that I can usually get around it if after clicking “Submit” in the dialog, I immediately click the Edit Option’s back button and save my changes. YMMV
Firmware: 2.1.5(2), Hardware Version:2.3 – DC
September 13, 2015 at 10:57 pm #40205
RayKeymasterIt turns out that Firmware 2.1.5 (including minor revision 2) has a bug that causes the boost time variable (for DC controller only) to be incorrectly reported as out of range. Specifically, the firmware internally stores the variable as a byte (ranging from 0 to 250), and that times 4 is the actual boost time. When it receives changes from the app, it’s supposed to be dividing that number by 4 for internal storage, but due to the bug it’s not dividing by 4 and consequently the error detection reports it’s out of range. This has already been fixed in firmware 2.1.6 (which will be released tonight or tomorrow). For the moment, the work-around is to change the Boost Time (under Advanced section) to 60 *every time* you make a change to any option. In other words, the actual boost time will be 240ms, but due to the bug, you have to manually divide that by 4 before submitting option changes, to avoid the out of range error.
Sorry about the issue, which we discovered just today.
September 16, 2015 at 1:47 pm #40245
ianfParticipantRay,
I’m also seeing this after upgrading to 2.1.6 compiled from git today (I’m using the Android AP). The controller is essentially an OS AC HW2.3. Is there an easy way to debug and see what input value it’s failing on?
Ian
September 16, 2015 at 8:39 pm #40252
RayKeymasterThis isn’t a known issue for OS AC version (we are only aware of ‘error saving options’ as an issue for OS DC version running firmware 2.1.5, due to the DC Boost Time option). So can you check to make sure if yours is AC or DC?
September 16, 2015 at 8:51 pm #40254
DennisParticipantRay, I updated to 2.1.6 and the “Please check input and try again” bug is resolved for me. Thanks.
Firmware: 2.1.6, Hardware Version:2.3 – DC
September 17, 2015 at 2:34 am #40263
ianfParticipantNever mind. To externally reference const (which are implicit static), the declaration needs to be as follows:
-prog_char op_max[] PROGMEM = {
+extern prog_char op_max[] PROGMEM = {In trying to figure this out, I had left in server.cpp
extern const char *op_max;instead of
extern const char op_max[];
which seems to be deferenced differently and messed up the maximum values.
September 18, 2015 at 3:20 pm #40273
ArndParticipantPlease don´t think I am crazy but although I haven´t upgraded to 2.1.6 yet, for some reason:
– the saving error bug is gone
– there is no boost time field in the options anymore
– my problem, that changes to the zimmerman parameters could not be saved is gone as well.All I did was a restart, but I doubt that this can be the reason.
Am I crazy? What happened?[even if I am crazy I am glad everything is working now]
September 19, 2015 at 7:47 am #40276
ArndParticipantand 2.1.7 ist working fine as well.
Thanks for the good support.
ArndSeptember 20, 2015 at 1:39 pm #40294
RayKeymaster@Arnd: Samer updated the UI/app script to not send the DC boost time variable for firmware 2.1.5, therefore, even if you haven’t upgraded your firmware, the bug is gone.
The reason to simply not send this variable is that firmware 2.1.5 has another bug, in that it ignores the DC boost time variable and uses a fixed 250ms instead. As a result, it doesn’t make any difference to change that variable at all. These bugs have all been fixed in firmware 2.1.6.
If you are wondering what role DC boost time plays: it controls how long the boost converter is turned on before opening each valve. The boost converter itself is set to a maximum voltage of 22V, so there is no danger in exceeding that voltage. But depending on the input voltage, the time it takes to reach that voltage may vary. For example, one feature of the DC controller is that you can even use USB power to drive the sprinkler valves (yes, you can try it, and it works). But USB power is only 5V, so it takes longer to boost that to 22V compared to a 9V input. In that case you may want to suitably increase the boost time to allow the extended charging time. The other reason to have a configurable DC boost time is that there are cases you might not want to enable the boost voltage, such as someone using a 12V power supply and drive 12V solenoids — in this case there is no need to boost the voltage to 22V. You can set the boost time to 0 to achieve this.
Now I think about it, it’s probably not necessary to have this as a configurable variable, and we may as well just remove it in the future.
-
AuthorPosts
- You must be logged in to reply to this topic.
OpenSprinkler › Forums › OpenSprinkler Mobile and Web App › Error when saving Options