-
Notifications
You must be signed in to change notification settings - Fork 73
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
Missing 'E' Segment (COM-11440) #15
Comments
I'm a little confused if this is a question/issue. Are you appending PB6/PB7 to pin_arduino.h? The copy of pins_arduino.h that is in this repo has had that for 3 years. Did you change something? Does 'Changing BOARD_TYPE to S7S' mean you changed line #39 of Serial_7_Segment_Display_Firmware.ino (#define DISPLAY_TYPE OPENSEGMENT) to use the S7S settings? |
Sorry for the lack of clarity. And yes, I've changed #define BOARD_TYPE to S7S. Which leads to better results than default but still seems to miss segment E. |
I think I've tracked the problem down. The build variant was set incorrectly to "standard", so pins_arduino.h wasn't being pulled from our custom hardware folder. I pushed a new version of boards.txt, if you want to try that out. Or just make sure this line says "ser7seg" not "standard". |
Wouldn't that require the directory
|
It does. https://github.com/sparkfun/Serial7SegmentDisplay/blob/master/firmware/hardware/Serial7Seg/variants/standard/pins_arduino.h Since you are running 1.6.4 you could try using the new board manager with this URL: https://raw.githubusercontent.com/sparkfun/Arduino_Boards/master/IDE_Board_Manager/package_sparkfun_index.json |
Same error with a fresh checkout. If I set |
@morganrallen is right. "standard" should be renamed "ser7seg". Those changes are pushed now. Bigger picture issue here is the hardware folder structure in general needs to be revised to match Arduino 1.6 standards. I'll keep this issue open until we resolve that. |
Right, I've had to add the |
So simply renaming everything hasn't worked. I got it back to a compiling state but Segment E remains dark. |
Another change pushed. Moved all hardware addon files to |
No dice. |
I cannot reproduce the error. What do you mean by the 'E' segment? Is it a single segment that is numbered in a figure in this section? Segment 5? If so, which digit(s)? |
Yes, E is the same as 5. |
This is an issue complained about in several comments across all of the 7 Segment product pages. Where everything appears normal, expect Segment 'E' on each digit does not display. I've verified the hardware is physically connected by switching pin assignments around and it does in fact light up.
Reading comments I've seen and tried several fixes.
Changing BOARD_TYPE to S7S.
Initially the text is totally scrambled, this change fixes it, except the E Segment.
Appending PB6/PB7 to pin_arduino.h
As I'm using the current code from this repo and (I think...) properly installing the hardware profile, this is taken care of for me.
I'm using Arduino 1.6.4
The text was updated successfully, but these errors were encountered: