New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Left hand Display #29

Closed
ltwin8 opened this Issue Jul 21, 2017 · 16 comments

Comments

4 participants
@ltwin8

ltwin8 commented Jul 21, 2017

The intro icon wich shows the iron with the buttons is the wrong direction. also the time to press both buttons are very short to exit solder mode...

Also very useful would be a permanent charge indicator for the battery, because most people I think use it with lipos and there would be nice to have for example a bar of pixels permanently on the display wich length relates to charge.... only 3 more menu items: CLOW (CELL LOW) GHIGH (CELL HIGH) CCOUNT (CELL COUNT)...

Great firmware for a rewrite until now :) 👍

@Ralim Ralim self-assigned this Jul 21, 2017

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 21, 2017

Owner

Hi,
I agree the left handed mode shows the icon reversed to what would be logical. I haven't had the time to sit down and draw in the flipped image as of yet sadly.

The intention is for pressing both buttons to be extremely hard to do accidentally, and require two hands (or a large thumb) to do correctly. Is this an issue for use of the iron? It is possible to add an extra delay to slow down the response time of the buttons, however, currently, the system is coded such that the response time of buttons anywhere in the use of the iron is the same.

I like the idea of a battery display, however, a large number of users of these irons use them on a non-battery supply. So requiring them to skip past multiple settings can be frustrating.

That said, most users use standard Lithium polymer batteries. If users are willing to live with set thresholds of like, minimum being > 3.V and cell maximum being >4.1V, then it would be possible to have the unit automatically detect the cell count and only require a setting to toggle the display of the battery status on and off?

Would that work acceptably for you?

Owner

Ralim commented Jul 21, 2017

Hi,
I agree the left handed mode shows the icon reversed to what would be logical. I haven't had the time to sit down and draw in the flipped image as of yet sadly.

The intention is for pressing both buttons to be extremely hard to do accidentally, and require two hands (or a large thumb) to do correctly. Is this an issue for use of the iron? It is possible to add an extra delay to slow down the response time of the buttons, however, currently, the system is coded such that the response time of buttons anywhere in the use of the iron is the same.

I like the idea of a battery display, however, a large number of users of these irons use them on a non-battery supply. So requiring them to skip past multiple settings can be frustrating.

That said, most users use standard Lithium polymer batteries. If users are willing to live with set thresholds of like, minimum being > 3.V and cell maximum being >4.1V, then it would be possible to have the unit automatically detect the cell count and only require a setting to toggle the display of the battery status on and off?

Would that work acceptably for you?

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 21, 2017

ltwin8 commented Jul 21, 2017

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 21, 2017

Owner

I like to try and respond quickly.

I'll look into making the detection a bit slower so its easier to hit the buttons. Might be a bit difficult with how the code is currently structured.

Yeah, that's a fair point about HV cells, completely mind blanked on them.
I'm really trying to avoid adding too many extra menu items, as it's already getting very long with options. I could add a menu option for "power source" that selects between 3-5S and DC so that it can automatically configure if it shows the battery status or not?

Owner

Ralim commented Jul 21, 2017

I like to try and respond quickly.

I'll look into making the detection a bit slower so its easier to hit the buttons. Might be a bit difficult with how the code is currently structured.

Yeah, that's a fair point about HV cells, completely mind blanked on them.
I'm really trying to avoid adding too many extra menu items, as it's already getting very long with options. I could add a menu option for "power source" that selects between 3-5S and DC so that it can automatically configure if it shows the battery status or not?

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 21, 2017

ltwin8 commented Jul 21, 2017

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 21, 2017

Owner

The input filter capacitors inside the iron are only rated to 25V input.
So you are technically overvolting the input capacitors. Also running the iron at the higher voltages also affects the voltage measurement as the overvolting of the ADC input does appear to degrade the input electronics.

While I can add the option for 6S, I don't want to encourage anyone to damage their irons.
So I will need to add a warning to the main page since exceeding the 25V input degrades the filtering capacitors and also affects the accuracy of the ADC readings.

I was thinking of using a combined menu, but the problem is for people who are using non-standard input sources, where the cutoff voltage may not align with normal LiPo cell voltages?

Owner

Ralim commented Jul 21, 2017

The input filter capacitors inside the iron are only rated to 25V input.
So you are technically overvolting the input capacitors. Also running the iron at the higher voltages also affects the voltage measurement as the overvolting of the ADC input does appear to degrade the input electronics.

While I can add the option for 6S, I don't want to encourage anyone to damage their irons.
So I will need to add a warning to the main page since exceeding the 25V input degrades the filtering capacitors and also affects the accuracy of the ADC readings.

I was thinking of using a combined menu, but the problem is for people who are using non-standard input sources, where the cutoff voltage may not align with normal LiPo cell voltages?

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 21, 2017

ltwin8 commented Jul 21, 2017

Ralim added a commit that referenced this issue Jul 22, 2017

Ralim added a commit that referenced this issue Jul 22, 2017

@reversebias

This comment has been minimized.

Show comment
Hide comment
@reversebias

reversebias Jul 23, 2017

Even though the caps are rated for 25V on the schematic, it's only the input filtering cap for the 3.9V switchmode converter. There's D4 (1N4148) and R30 (150R) in series with the supply to the caps, which drop about 2.5V under normal conditions. I measured 22.6V on the cap with a 25.2V input voltage. Although 25V caps are a bit marginal here, I don't think that using a 6S lipo is a safety risk.

reversebias commented Jul 23, 2017

Even though the caps are rated for 25V on the schematic, it's only the input filtering cap for the 3.9V switchmode converter. There's D4 (1N4148) and R30 (150R) in series with the supply to the caps, which drop about 2.5V under normal conditions. I measured 22.6V on the cap with a 25.2V input voltage. Although 25V caps are a bit marginal here, I don't think that using a 6S lipo is a safety risk.

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 23, 2017

Owner

Hi,
While I agree with you that you should be fine. I need to put the warning there, as its running the unit out-of-spec. This is why I added the 6S menu option, but I also added the warning to the main page.
Anway, can we please move the discussion on battery stuff over to #30 .

Owner

Ralim commented Jul 23, 2017

Hi,
While I agree with you that you should be fine. I need to put the warning there, as its running the unit out-of-spec. This is why I added the 6S menu option, but I also added the warning to the main page.
Anway, can we please move the discussion on battery stuff over to #30 .

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 24, 2017

Owner

@ltwin8 Can you please clarify with the buttons to exist soldering mode, did you mean that it was too easy to exit soldering more, or that you would like it to be easier to get the double button press (ie exit easier)?

Owner

Ralim commented Jul 24, 2017

@ltwin8 Can you please clarify with the buttons to exist soldering mode, did you mean that it was too easy to exit soldering more, or that you would like it to be easier to get the double button press (ie exit easier)?

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 24, 2017

ltwin8 commented Jul 24, 2017

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 24, 2017

Owner

I can't really make it a single button action, but I will try and make the time it uses for detection of the double button press a bit longer. If your thumb is large enough it should be fairly easy to push both with one hand?

Owner

Ralim commented Jul 24, 2017

I can't really make it a single button action, but I will try and make the time it uses for detection of the double button press a bit longer. If your thumb is large enough it should be fairly easy to push both with one hand?

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 24, 2017

yes I think with a bit training it would be easier to exit it with one hand.

maybe I found an other thing today: when I adjust temperature in soldering mode (Left hand) the temperature rises with the "usb-end" button, is that right?

Best regards

ltwin8 commented Jul 24, 2017

yes I think with a bit training it would be easier to exit it with one hand.

maybe I found an other thing today: when I adjust temperature in soldering mode (Left hand) the temperature rises with the "usb-end" button, is that right?

Best regards

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 24, 2017

Owner

See how you go with it currently trying left handed mode. I'll try and extend the time slightly in the next version.

And yep that is a bug :/ Ill fix that too in 1.16

Owner

Ralim commented Jul 24, 2017

See how you go with it currently trying left handed mode. I'll try and extend the time slightly in the next version.

And yep that is a bug :/ Ill fix that too in 1.16

@ltwin8

This comment has been minimized.

Show comment
Hide comment
@ltwin8

ltwin8 Jul 24, 2017

Yes I try left hand mode, with right hand I can't solder.
its not a big problem, its like complain on a very high level.

by the way, the way the battery status is shown is great, it don't overlay important informations like temperature and soldering status and it is good readable outside

ltwin8 commented Jul 24, 2017

Yes I try left hand mode, with right hand I can't solder.
its not a big problem, its like complain on a very high level.

by the way, the way the battery status is shown is great, it don't overlay important informations like temperature and soldering status and it is good readable outside

Ralim pushed a commit that referenced this issue Jul 24, 2017

Ben V. Brown Ben V. Brown
Fix buttons in left handed mode when user is changing the temperature.
As pointed out in #29

Clear the leftover temperature unit on the sleep screen.
@joric

This comment has been minimized.

Show comment
Hide comment
@joric

joric Jul 26, 2017

Hey, @reversebias ! Nice to see you here, I'm currently trying to build your mitosis keyboard out of EasyEDA PCB's :) As for this left hand feature I suggest using accelerometer for autorotation. Current accelerometer API is poor but all you really need (in 2.17) is to check for a single bit, e.g. gY_value.Byte.hi>0x7f ? -1 : 1; set corresponding screen orientation and then call Init_Oled(). I've used it here https://github.com/joric/ts100tris/tree/autorotate Autorotation might be annoying but it works for phones and people kind of expect that from the device that has a 3-axis accelerometer.

joric commented Jul 26, 2017

Hey, @reversebias ! Nice to see you here, I'm currently trying to build your mitosis keyboard out of EasyEDA PCB's :) As for this left hand feature I suggest using accelerometer for autorotation. Current accelerometer API is poor but all you really need (in 2.17) is to check for a single bit, e.g. gY_value.Byte.hi>0x7f ? -1 : 1; set corresponding screen orientation and then call Init_Oled(). I've used it here https://github.com/joric/ts100tris/tree/autorotate Autorotation might be annoying but it works for phones and people kind of expect that from the device that has a 3-axis accelerometer.

@Ralim

This comment has been minimized.

Show comment
Hide comment
@Ralim

Ralim Jul 26, 2017

Owner

Hi, I agree that auto rotation could be a nice thing to add.
I haven't fleshed out the full API for the accelerometer yet as I didn't need to 😄
It's trivial to add this as a feature, so I should be able to soon.

I will probably update the setting to be 3 options, so you can lock the orientation or have it automatically change.

Owner

Ralim commented Jul 26, 2017

Hi, I agree that auto rotation could be a nice thing to add.
I haven't fleshed out the full API for the accelerometer yet as I didn't need to 😄
It's trivial to add this as a feature, so I should be able to soon.

I will probably update the setting to be 3 options, so you can lock the orientation or have it automatically change.

@Ralim Ralim closed this in 1cbcba9 Jul 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment