Skip to content
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

Residual border on MagTag #55

Closed
caternuson opened this issue Nov 18, 2021 · 7 comments
Closed

Residual border on MagTag #55

caternuson opened this issue Nov 18, 2021 · 7 comments

Comments

@caternuson
Copy link
Contributor

Re this thread:
https://forums.adafruit.com/viewtopic.php?f=22&t=185293

CircuitPython usage does not have this behavior. The Learn guide Arduino example shows the border as well.

Can recreate same behavior with this example:

#include "Adafruit_ThinkInk.h"
#include <Fonts/FreeSans9pt7b.h>

ThinkInk_290_Grayscale4_T5 display(EPD_DC, EPD_RESET, EPD_CS, -1, -1);

void setup() {
  Serial.begin(9600);
  while (!Serial) delay(10);
  Serial.println("Adafruit EPD full update test in mono & grayscale");

  display.begin(THINKINK_GRAYSCALE4);
  display.clearBuffer();
  display.setFont(&FreeSans9pt7b);
  display.setTextSize(1);
  display.setTextColor(EPD_BLACK);
  display.setCursor(10, 30);
  display.print("HELLO WORLD");
  display.display();
}

void loop() {
}
@ladyada
Copy link
Member

ladyada commented Nov 18, 2021

this would be in one of the init config bytes. its not documented so would have to sorta compare the bytes

@caternuson
Copy link
Contributor Author

I tried using the CP config, since there are some differences:
https://github.com/adafruit/circuitpython/blob/932131b4ff4b95066a872b5b299a84e80b7c45d3/ports/espressif/boards/adafruit_magtag_2.9_grayscale/board.c#L45-L52
replacing this:

static const uint8_t ti_290t5_gray4_init_code[] {
IL0373_POWER_SETTING, 5, 0x03, 0x00, 0x2b, 0x2b, 0x13,
IL0373_BOOSTER_SOFT_START, 3, 0x17, 0x17, 0x17,
IL0373_POWER_ON, 0,
0xFF, 200,
IL0373_PANEL_SETTING, 1, 0x3F,
IL0373_PLL, 1, 0x3C,
IL0373_VCM_DC_SETTING, 1, 0x12,
IL0373_CDI, 1, 0x97,
0xFE // EOM
};

with:

static const uint8_t ti_290t5_gray4_init_code[] {
  0x01, 5, 0x03, 0x00, 0x2b, 0x2b, 0x13,
  0x06, 3, 0x17, 0x17, 0x17,
  0x04, 0,
  0xFF, 200,
  0x00, 1, 0x7f,
  0x50, 1, 0x97,
  0x30, 1, 0x3C,
  0x61, 3, 0x80, 0x01, 0x28,
  0x82, 1, 0x12,
  0xFF, 50,
  0xFE // EOM
};

But no change. Border is still there.

@caternuson
Copy link
Contributor Author

I was wrong about CP not doing this. It does. You can see it here from the weather guide:

image

And I've recreated that again locally to verify.

As mentioned in the forum thread, this is happening outside the addressable display area.

@caternuson
Copy link
Contributor Author

This appears to be a feature of the display. There is a border data selection, but unfortunately it is not documented beyond specifying which bits they are:
image

Testing all combos resulted in:

VBD[1:0]:
00b dark gray
01b dark gray
10b black
11b light gray (default)

So it appears there is no way to turn the border "off".

Closing this for now. Can reopen if more info becomes available on how to set it differently.

@OneOfTheInfiniteMonkeys

To help with some linked items....
https://forums.adafruit.com/viewtopic.php?f=22&t=185293
adafruit/Adafruit_CircuitPython_IL0373#26
adafruit/Adafruit_CircuitPython_IL0373#11

You are not alone.... ;-)
The last post seemt to imply it's fixed, if it is its opaque ( :-o ) to me ;-)

@lindes
Copy link

lindes commented Aug 19, 2023

To help with some linked items....

Ooh, thanks for those.

You are not alone.... ;-)

Haha, nope. :)

The last post seemt to imply it's fixed, if it is its opaque ( :-o ) to me ;-)

It looks like the fix (which I'd actually chalk up to the pull request of your second link, but that's mentioned in the last link, so fair enough) is in the Adafruit_CircuitPython_IL0373 library... which... I'm guessing the MagTag library uses indirectly? (I can't see a direct usage of it, and I'm failing to track it down... I think it comes from CircuitPython's board.DISPLAY somehow, but even that I failed to track all the way down... so, perhaps I'm mistaken, but hopefully this is indeed the case.)

Given all of that, is it possible to get this new border flag from the IL0373 library to be exposed somehow? If so, could we re-open this issue to track that, or shall I submit a new one, or... ?

Thanks!

@OneOfTheInfiniteMonkeys

Hi, well as you will see from the posts in link adafruit/Adafruit_CircuitPython_IL0373#11, dated 09 Aug 2021, things seemed to have happened. aka comments per Tanweet and Makermelissa.

Everyone seemed to think it could be changed, some even commented it should be changed.

Unfortunately, if there was a change, from my perspective the changes are so opaque, I don't know how to use them.
:-(
Perhaps one day it, will get clearer ;-).

So open source, If I could have figured out or found instructions to compile on a Windows machine, I would have. Then I would not be working on my projects :-(

At the moment I have a Crash Crikey! issue in CircuitPythons >8.0.5 distracting me from anything else.
See here for the current (at time of writing) project https://github.com/OneOfTheInfiniteMonkeys/MTDL where its causing me pain. (I rasied an MQQT % fill display bug issue for the vertical bar displays for that project, it's fixed now - Yeah.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants