search

stanleyhuangyc / MultiLCD

Arduino LCD library supporting multiple types of LCD shields and modules
112 stars 70 forks source link

Full set of font sizes not working with Microduino-Core #4

Open dweston opened 10 years ago

dweston commented 10 years ago

In prep for using a Microduino-OLED card with a newly purchased Freematics OBD-II adapter I’m presently feeling my way around how to use your MultiLCD and MiniLCD libraries your with the OLED hooked up to a Microduino-Core card.

When calling the lcd.printLong function the MEDIUM and LARGE font sizes do not work correctly. Calling the MEDIUM sized font setting:

lcd.setFont(FONT_SIZE_MEDIUM);

results in the SMALL sized font being displayed, and calling the LARGE font size:

lcd.setFont(FONT_SIZE_LARGE);

results in the XLARGE sized font being displayed.

Calling the SMALL and XLARGE font sizes display correctly.

When calling the lcd.print function all font size settings default to SMALL.

If I replace the Microduino-Core card with a Microduino-Core+ card the libraries display all font sizes as intended.

stanleyhuangyc commented 10 years ago 
Please modify one line in MultiLCD.h.
  Make sure
  #define MEMORY_SAVING
  is marked out.
  Stanley
  On 2014/2/27 12:05, Dion Weston wrote:

  In prep for using a Microduino-OLED card with a newly purchased
    Freematics OBD-II adapter I’m presently feeling my way around
    how to use your MultiLCD and MiniLCD libraries your with the
    OLED hooked up to a Microduino-Core card.
  When calling the lcd.printLong function the MEDIUM and LARGE
    font sizes do not work correctly. Calling the MEDIUM sized font
    setting:
  lcd.setFont(FONT_SIZE_MEDIUM);
  results in the SMALL sized font being displayed, and calling
    the LARGE font size:
  lcd.setFont(FONT_SIZE_LARGE);
  results in the XLARGE sized font being displayed.
  Calling the SMALL and XLARGE font sizes display correctly.
  When calling the lcd.print function all font size settings
    default to SMALL.
  If I replace the Microduino-Core card with a Microduino-Core+
    card the libraries display all font sizes as intended.
  —
    Reply to this email directly or view
      it on GitHub.
dweston commented 10 years ago

Only one mention of '#define MEMORY_SAVING’ in MultiLCD.h.

if !defined(AVR_ATmega2560) && !defined(AVR_ATmega1280) && !defined(AVR_ATmega644P) && !defined(SAM3X8E)

//#define MEMORY_SAVING

endif

And this line is already commented out.

However the mention of this line in MicroLCD.h is not commented out and when I recompile with it marked out the full font range now works with the smaller processor.

Thx

Dion

On 27 Feb 2014, at 15:53 , Stanley Huang notifications@github.com wrote:

Please modify one line in MultiLCD.h. Make sure

define MEMORY_SAVING

is marked out. Stanley On 2014/2/27 12:05, Dion Weston wrote:

In prep for using a Microduino-OLED card with a newly purchased Freematics OBD-II adapter I’m presently feeling my way around how to use your MultiLCD and MiniLCD libraries your with the OLED hooked up to a Microduino-Core card. When calling the lcd.printLong function the MEDIUM and LARGE font sizes do not work correctly. Calling the MEDIUM sized font setting: lcd.setFont(FONT_SIZE_MEDIUM); results in the SMALL sized font being displayed, and calling the LARGE font size: lcd.setFont(FONT_SIZE_LARGE); results in the XLARGE sized font being displayed. Calling the SMALL and XLARGE font sizes display correctly. When calling the lcd.print function all font size settings default to SMALL. If I replace the Microduino-Core card with a Microduino-Core+ card the libraries display all font sizes as intended. — Reply to this email directly or view it on GitHub. — Reply to this email directly or view it on GitHub.

stanleyhuangyc commented 10 years ago 
Sorry, it's MicroLCD.h
  On 2014/2/27 13:48, Dion Weston wrote:
Only one mention of '#define MEMORY_SAVING’ in
  MultiLCD.h.

  #if !defined(__AVR_ATmega2560__) &&
  !defined(__AVR_ATmega1280__) &&
  !defined(__AVR_ATmega644P__) && !defined(__SAM3X8E__)

  //#define MEMORY_SAVING

  #endif

  And this line is already commented out.

  However the mention of this line in MicroLCD.h is not commented
  out and when I recompile with it marked out the full font range
  now works with the smaller processor.

  Thx

  Dion

  On 27 Feb 2014, at 15:53 , Stanley Huang
  <notifications@github.com> wrote:

  > 
  > Please modify one line in MultiLCD.h.

  > Make sure

  > #define MEMORY_SAVING

  > is marked out.

  > Stanley

  > On 2014/2/27 12:05, Dion Weston wrote:

  > 
  > In prep for using a Microduino-OLED card with a newly
  purchased

  > Freematics OBD-II adapter I’m presently feeling my way around

  > how to use your MultiLCD and MiniLCD libraries your with the

  > OLED hooked up to a Microduino-Core card.

  > When calling the lcd.printLong function the MEDIUM and LARGE

  > font sizes do not work correctly. Calling the MEDIUM sized
  font

  > setting:

  > lcd.setFont(FONT_SIZE_MEDIUM);

  > results in the SMALL sized font being displayed, and calling

  > the LARGE font size:

  > lcd.setFont(FONT_SIZE_LARGE);

  > results in the XLARGE sized font being displayed.

  > Calling the SMALL and XLARGE font sizes display correctly.

  > When calling the lcd.print function all font size settings

  > default to SMALL.

  > If I replace the Microduino-Core card with a Microduino-Core+

  > card the libraries display all font sizes as intended.

  > —

  > Reply to this email directly or view

  > it on GitHub.

  > —

  > Reply to this email directly or view it on GitHub.

  >
  —
    Reply to this email directly or view
      it on GitHub.
dweston commented 10 years ago

Thx

I note that with that line uncommented the memory requirements are circa 12Kbs. If its commented out the memory used rises to 15Kb.

That’s worth knowing if I run out of memory some time.

On 27 Feb 2014, at 17:03 , Stanley Huang notifications@github.com wrote:

Sorry, it's MicroLCD.h On 2014/2/27 13:48, Dion Weston wrote: Only one mention of '#define MEMORY_SAVING’ in MultiLCD.h.

if !defined(AVR_ATmega2560) &&

!defined(AVR_ATmega1280) && !defined(AVR_ATmega644P) && !defined(SAM3X8E)

//#define MEMORY_SAVING

endif

And this line is already commented out.

However the mention of this line in MicroLCD.h is not commented out and when I recompile with it marked out the full font range now works with the smaller processor.

Thx

Dion

On 27 Feb 2014, at 15:53 , Stanley Huang notifications@github.com wrote:

Please modify one line in MultiLCD.h.

Make sure

define MEMORY_SAVING

is marked out.

Stanley

On 2014/2/27 12:05, Dion Weston wrote:

In prep for using a Microduino-OLED card with a newly purchased

Freematics OBD-II adapter I’m presently feeling my way around

how to use your MultiLCD and MiniLCD libraries your with the

OLED hooked up to a Microduino-Core card.

When calling the lcd.printLong function the MEDIUM and LARGE

font sizes do not work correctly. Calling the MEDIUM sized font

setting:

lcd.setFont(FONT_SIZE_MEDIUM);

results in the SMALL sized font being displayed, and calling

the LARGE font size:

lcd.setFont(FONT_SIZE_LARGE);

results in the XLARGE sized font being displayed.

Calling the SMALL and XLARGE font sizes display correctly.

When calling the lcd.print function all font size settings

default to SMALL.

If I replace the Microduino-Core card with a Microduino-Core+

card the libraries display all font sizes as intended.

—

Reply to this email directly or view

it on GitHub.

—

Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub. — Reply to this email directly or view it on GitHub.

stanleyhuangyc commented 10 years ago 
It's designed for saving memory.
  On 2014/2/27 14:28, Dion Weston wrote:
Thx

  I note that with that line uncommented the memory requirements are
  circa 12Kbs. If its commented out the memory used rises to 15Kb.

  That’s worth knowing if I run out of memory some time.

  On 27 Feb 2014, at 17:03 , Stanley Huang
  <notifications@github.com> wrote:

  > 
  > Sorry, it's MicroLCD.h

  > On 2014/2/27 13:48, Dion Weston wrote:

  > Only one mention of '#define MEMORY_SAVING’ in

  > MultiLCD.h.

  > 
  > #if !defined(__AVR_ATmega2560__) &&

  > !defined(__AVR_ATmega1280__) &&

  > !defined(__AVR_ATmega644P__) && !defined(__SAM3X8E__)

  > 
  > //#define MEMORY_SAVING

  > 
  > #endif

  > 
  > And this line is already commented out.

  > 
  > However the mention of this line in MicroLCD.h is not
  commented

  > out and when I recompile with it marked out the full font
  range

  > now works with the smaller processor.

  > 
  > Thx

  > 
  > Dion

  > 
  > On 27 Feb 2014, at 15:53 , Stanley Huang

  > <notifications@github.com> wrote:

  > 
  > > 
  > > Please modify one line in MultiLCD.h.

  > 
  > > Make sure

  > 
  > > #define MEMORY_SAVING

  > 
  > > is marked out.

  > 
  > > Stanley

  > 
  > > On 2014/2/27 12:05, Dion Weston wrote:

  > 
  > > 
  > > In prep for using a Microduino-OLED card with a newly

  > purchased

  > 
  > > Freematics OBD-II adapter I’m presently feeling my way
  around

  > 
  > > how to use your MultiLCD and MiniLCD libraries your with
  the

  > 
  > > OLED hooked up to a Microduino-Core card.

  > 
  > > When calling the lcd.printLong function the MEDIUM and
  LARGE

  > 
  > > font sizes do not work correctly. Calling the MEDIUM
  sized

  > font

  > 
  > > setting:

  > 
  > > lcd.setFont(FONT_SIZE_MEDIUM);

  > 
  > > results in the SMALL sized font being displayed, and
  calling

  > 
  > > the LARGE font size:

  > 
  > > lcd.setFont(FONT_SIZE_LARGE);

  > 
  > > results in the XLARGE sized font being displayed.

  > 
  > > Calling the SMALL and XLARGE font sizes display
  correctly.

  > 
  > > When calling the lcd.print function all font size
  settings

  > 
  > > default to SMALL.

  > 
  > > If I replace the Microduino-Core card with a
  Microduino-Core+

  > 
  > > card the libraries display all font sizes as intended.

  > 
  > > —

  > 
  > > Reply to this email directly or view

  > 
  > > it on GitHub.

  > 
  > > —

  > 
  > > Reply to this email directly or view it on GitHub.

  > 
  > >

  > —

  > Reply to this email directly or view

  > it on GitHub.

  > —

  > Reply to this email directly or view it on GitHub.

  >
  —
    Reply to this email directly or view
      it on GitHub.
dweston commented 10 years ago

Indeed I realised that. What was noteworthy was the amount of memory saved for a fairly limited loss of functionality.

3Kb is a lot in the ATmega328 scheme of things.

Dion

On 27 Feb 2014, at 17:32 , Stanley Huang notifications@github.com wrote:

It's designed for saving memory. On 2014/2/27 14:28, Dion Weston wrote: Thx

I note that with that line uncommented the memory requirements are circa 12Kbs. If its commented out the memory used rises to 15Kb.

That’s worth knowing if I run out of memory some time.

On 27 Feb 2014, at 17:03 , Stanley Huang notifications@github.com wrote:

Sorry, it's MicroLCD.h

On 2014/2/27 13:48, Dion Weston wrote:

Only one mention of '#define MEMORY_SAVING’ in

MultiLCD.h.

if !defined(AVR_ATmega2560) &&

!defined(AVR_ATmega1280) &&

!defined(AVR_ATmega644P) && !defined(SAM3X8E)

//#define MEMORY_SAVING

endif

And this line is already commented out.

However the mention of this line in MicroLCD.h is not commented

out and when I recompile with it marked out the full font range

now works with the smaller processor.

Thx

Dion

On 27 Feb 2014, at 15:53 , Stanley Huang

notifications@github.com wrote:

Please modify one line in MultiLCD.h.

Make sure

define MEMORY_SAVING

is marked out.

Stanley

On 2014/2/27 12:05, Dion Weston wrote:

In prep for using a Microduino-OLED card with a newly

purchased

Freematics OBD-II adapter I’m presently feeling my way around

how to use your MultiLCD and MiniLCD libraries your with the

OLED hooked up to a Microduino-Core card.

When calling the lcd.printLong function the MEDIUM and LARGE

font sizes do not work correctly. Calling the MEDIUM sized

font

setting:

lcd.setFont(FONT_SIZE_MEDIUM);

results in the SMALL sized font being displayed, and calling

the LARGE font size:

lcd.setFont(FONT_SIZE_LARGE);

results in the XLARGE sized font being displayed.

Calling the SMALL and XLARGE font sizes display correctly.

When calling the lcd.print function all font size settings

default to SMALL.

If I replace the Microduino-Core card with a Microduino-Core+

card the libraries display all font sizes as intended.

—

Reply to this email directly or view

it on GitHub.

—

Reply to this email directly or view it on GitHub.

—

Reply to this email directly or view

it on GitHub.

—

Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub. — Reply to this email directly or view it on GitHub.

talofer99 commented 10 years ago

I'm working on an OBD I (correct . 1 not 2) output of data for my car. I found a OLED screen and your library works great with it, defiantly way faster then any thing else - so thanks for that. My Q is (and I just could not find any better place to ask it) can you help me out on the "how to" make the digits font even bigger. the reason is that I want to display the speed and bellow "bars" that will represent the RPM value ... I need bigger digits. Any help will be highly appreciated. Thanks Tal.