At a quick glance, NarsilM seems to be using a uint8_t for its temperature value, so it can’t represent anything colder than freezing. It also appears to use a uint16_t for the initial raw value, from which it subtracts 275, which could set the value to 65,5XX during cold weather.
To fix only the negative temperature issues, it might work to simply use signed 16-bit values for all of this, and maybe add some checks to make sure it’s above 0. But I’m just guessing.
Hi Tom,
some suggestions for the next version of NarsilM.
I would suggest to double the time to enter settings. From 8 seconds to 16 seconds or 20.
(Edit: The 20s would interfere with the jammed button protection)
And change the clicks for RAMPING and MODES in the settings.
I think he has a point making NarsilM more robust against wrong operation.
If I handle someone the light I tell “don’t press for longer than 8 seconds”.
I think extending the time can help. The only backside would be if some switches often between MODES and RAMPING. But I guess most stay in one UI.
I guess you used 8sec to be able to switch quickly between RAMPING and MODES.
The sequence to change is:
Hold the button till it blinks the first settings, then one or two clicks and then hold the button till the lamp confirm with four blinks.
In retrospect I think it would also better to change the first setting to
1 click = RAMPING
2 clicks = MODES
Today it’s reversed.
Only for the reason RAMPING is mostly used and people try to switch off the lamp if it starts blinking.
changing the 8 to 20 seconds is just one line in the Narsil.c file
CONFIG hold - if it is not locked out or lock-out was just exited on this hold
//———————————————————————————————————————————-
if ((byLockOutSet 0) && !momentaryState)
{
if ((!ramping && (wPressDuration CONFIG_ENTER_DUR) && (fastClicks != 2) && (modeIdx != BATT_CHECK_MODE))
||
(ramping && (wPressDuration == 500))) // 8 secs in ramping mode
same for Modes to extend 2.5 to more in the setup file #define CONFIG_ENTER_DUR 160 // Config mode entry hold time - 160=2.5s, 128=2s
I changed the register and PB port in the register settings but it does not work
It seems I have to change something in the channels.h file as well, but that code does not really make sense to me so I cant change it
I’m almost certain the answer is no. When you set the step down in ramping it automatically carries over to the mode sets and vice versa. So I don’t think there’s any way you can separate them. Besides, I’m not sure why you would want different step-downs between ramping and mode sets in the first place.
The reason I ask is, I found an older entry on the german forum where it should work.
He said he has in one moon aktiv in the other not.
I never tested it and have also doubts, so I asked here
Looking at the code, it has only one value which is shared across both. Tom documented it in the comment for SaveConfig():
/**************************************************************************************
* SaveConfig - save the current mode with config settings
* ==========
* Central method for writing (with wear leveling)
*
* config1 - 1st byte of stored configuration settings:
* bits 0-2: mode index (0..7), for clicky mode switching
* bits 3-6: selected mode set (0..11)
* bit 7: ramping mode
*
* config2 - 2nd byte of stored configuration settings:
* bit 0: mode ordering, 1=hi to lo, 0=lo to hi
* bit 1: mode memory for the e-switch - 1=enabled, 0=disabled
* bit 2-4: moonlight level, 1-7 enabled on the PWM value of 1-7, 0=disabled
* bits 5-7: stepdown: 0=disabled, 1=thermal, 2=60s, 3=90s, 4=2min, 5=3min, 6=5min, 7=7min
*
* config3 - 3rd byte of stored configuration settings:
* bit 0: 1: Do OFF time mode memory on power switching (tailswitch), 0: disabled
* bit 1: On Board LED support - 1=enabled, 0=disabled
* bit 2: Locator LED feature (ON when light is OFF) - 1=enabled, 0=disabled
* bit 3: BVLD LED Only - 1=BVLD only w/onboard LED, 0=both primary and onboard LED's are used
* bit 4: 1: moonlight mode - 1=enabled, 0=disabled
* bit 5-6: blinky mode config: 1=strobe only, 2=all blinkies, 0=disable
*
**************************************************************************************/
I thought about exploiting the lock timing to be infinite or removed
If the firmware does not lock the mode in after 1.2s it would all the time ready to change modes
ramping does not use the 1.2s mode lock timer its only used in modes set
I cant program so I do not know what ecxactly to change to get this timer set to infinite or disable it
so in ramping side switch has normal operation including putting the light in standby