Well the inevitable bug turned up .
When the alarm is activated, deactivating it using the IR sensor can cause the clock to lock up and not respond to the proximity sensor until powered down and reset (either by removing the processor from the board or waiting for the backup capacitor to discharge).
The IR sensor has 3 levels of distance sensing, normally only 2 of these are used but the third was at one point intended to be used for the alarm disable. In the final code it wasn't but the link to it wasn't remmed out.
If you switch off the alarm by holding your hand above the sensor at the top or middle of its sensing range everything will work fine, however if you hold it too close to the clock it will activate the unwanted 3rd level and lock up.
If you have access to a pic programmer I've attached an updated hex file, if not please contact me to arrange a replacement processor.
Sorry about that - we really did test everything repeatedly but there's always (at least) one that gets away.
Note that this only affects the first batch of kits.
Alarm lockup firmware bug
Alarm lockup firmware bug
- Attachments
-
- 5-tube clock v1.1 piccode.rar
- (7.73 KiB) Downloaded 1295 times
Re: Alarm lockup firmware bug
Version with added AM/PM DP indicator in the minutes-tens tube.
- Attachments
-
- 5-tube clock v1.1 PM DP piccode.rar
- (7.79 KiB) Downloaded 1290 times
Re: Alarm lockup firmware bug
...and just in case anyone wants to play with the code, here is the .asm file. I think most of the profanities have been edited out of the comments
- Attachments
-
- 5-tube clock code Oct14.rar
- (23.72 KiB) Downloaded 1365 times
Re: Alarm lockup firmware bug
Tony, on the supplied assembler source. What assemble did you use to build this code?Tony wrote:...and just in case anyone wants to play with the code, here is the .asm file. I think most of the profanities have been edited out of the comments
Re: Alarm lockup firmware bug
Just the standard microchip MPLAB IDE, version 8.90. The configuration bits should already be set in the code, processor type is 16f1827 and use a PICKIT3 to program.
Re: Alarm lockup firmware bug
As requested, a version which doesn't blank the leading zero in 12h mode
- Attachments
-
- 5-tube clock 12h mod piccode.rar
- (7.75 KiB) Downloaded 1279 times
Re: Alarm lockup firmware bug
Hello Tony, could you provide the latest asm updated version? I would like to start from it to add the GPS fix code...
Thank you for that !
Thank you for that !
Re: Alarm lockup firmware bug
Hello,
please find here the fixed version (asm + hex) of the clock.
This version fixes the GPS issue (clock crash when GPS is not ready to transmit correct hhmmss strings)
I tested it, and it definitely fixes the issue!
do not hesitate to try it ! and give me back your comments
please find here the fixed version (asm + hex) of the clock.
This version fixes the GPS issue (clock crash when GPS is not ready to transmit correct hhmmss strings)
I tested it, and it definitely fixes the issue!
do not hesitate to try it ! and give me back your comments
- Attachments
-
- 5-tube_clock_code_apr18_correction_v1.zip
- (33.61 KiB) Downloaded 838 times
Re: Alarm lockup firmware bug
Hi,
Thanks - I haven't checked the forums for a few days. I'll give it a try over the weekend,
Tony.
Thanks - I haven't checked the forums for a few days. I'll give it a try over the weekend,
Tony.
Re: Alarm lockup firmware bug
Hello,
here is a new version of the code. I hope this is the final one!
Here is a description of the changes:
- Added parts of the GPS code that is taken from the "TimeFlies" kit ( I added the control of a valid GPS string : control of the character "A")
- Fixed the bug of the BST time that was not taken into account when connecting a GPS
- Fixed several bugs linked to the use of variables in bank1 (some variables in Bank1 were used in the code without the instruction "Banksel1" before them)
You can find all my fixes by searching "EBD" in the comments
This version is under test since the 03rd of July and I saw no default for the moment
here is a new version of the code. I hope this is the final one!
Here is a description of the changes:
- Added parts of the GPS code that is taken from the "TimeFlies" kit ( I added the control of a valid GPS string : control of the character "A")
- Fixed the bug of the BST time that was not taken into account when connecting a GPS
- Fixed several bugs linked to the use of variables in bank1 (some variables in Bank1 were used in the code without the instruction "Banksel1" before them)
You can find all my fixes by searching "EBD" in the comments
This version is under test since the 03rd of July and I saw no default for the moment
- Attachments
-
- 5-tube clock code_july2018.zip
- (34.21 KiB) Downloaded 838 times