Menu for Display

Hi Oliver,
here you are the code :slight_smile:

#include <Time.h>
#include <TimeAlarms.h>
#include <LiquidCrystal.h>	// ensure that the include path is set
#include "M2tk.h"
#include <VirtualWire.h>
#include <Wire.h>
#include <DS1307.h>
#include <EEPROM.h>
#if defined(ARDUINO) && ARDUINO >= 100
#include "Arduino.h"
#else
#include "WProgram.h"
#endif

//Variabili Globali
char dev;
char testo[15];

// usata per mandare il messaggio
char messaggio[15];
// usato nell'index del close del keypad
int ind = 0;
// usato nel keypad
char Tasto;
// usato nel keyoad
unsigned long UltimoTasto = 0;
// usato nel nella procedura seriale
String LeggiStringa;
byte memoria[105];

TimeElements tm0start;
TimeElements tm0stop;
TimeElements tm1start;
TimeElements tm1stop;
TimeElements tm2start;
TimeElements tm2stop;
TimeElements tm3start;
TimeElements tm3stop;
TimeElements tm4start;
TimeElements tm4stop;
TimeElements tm5start;
TimeElements tm5stop;
TimeElements tm6start;
TimeElements tm6stop;
TimeElements tm7start;
TimeElements tm7stop;



void LeggiEeprom()
{
  int ind;
   for (ind =0; ind <= 105 ; ind++)
      {
        memoria[ind] = EEPROM.read(ind);
      }   
}

void Timer()
{
 
     tm0start.Hour =   memoria[2];
     tm0start.Minute = memoria[3];
     tm0start.Second = 00;
     tm0start.Day =   memoria[4];
     tm0start.Month = memoria[5];
     tm0start.Year = ((memoria[6]+2000) - 1970);
  
     Alarm.triggerOnce(makeTime(tm0start), StartTimer0);
     
     
     tm0stop.Hour =   memoria[7];
     tm0stop.Minute = memoria[8];
     tm0stop.Second = 00;
     tm0stop.Day =   memoria[9];
     tm0stop.Month = memoria[10];
     tm0stop.Year = ((memoria[11]+2000) - 1970);
  
    Alarm.triggerOnce(makeTime(tm0stop), StopTimer0);
     
  
  if ( memoria[13] != 0 )
  {
     
  }
  
  if ( memoria[26] != 0 )
  {
     
  }
  
  if ( memoria[39] != 0 )
  {
     
  }
  
  if ( memoria[52] != 0 )
  {
     
  }
  
  if ( memoria[65] != 0 )
  {
     
  }
  
  if ( memoria[78] != 0 )
  {
     
  }
  
  if ( memoria[91] != 0 )
  {
     
  }
  
}


void StartTimer0()
{
  messaggio[0]='1';
  messaggio[1]='0';
  messaggio[2]='#';
  TxMsg(messaggio);
  Serial.println("Start Timer 0");
}

void StopTimer0()
{
  messaggio[0]='*';
  messaggio[1]='0';
  messaggio[2]='#';
  TxMsg(messaggio);
  Serial.println("Stop Timer 0");
}

void DefinizionePin()
{
  //Piedini Utilizzati per Gestire LCD  
  pinMode(A0, INPUT);
  pinMode(A1, INPUT);
  pinMode(A2, INPUT);
  pinMode(A3, INPUT);
  pinMode(A4, INPUT);
  pinMode(A5, INPUT);
  pinMode(13, OUTPUT);
}

void Comunicazione()
{
   Wire.begin();         // Inizializzo la Comunicazione I2C
   Serial.begin(9600);	 // Inizializzo la Seriale
   vw_setup(1200);	 // Inizializzo al Comunicazione TTL su Virtual Wire
   vw_rx_start();        // Start the receiver PLL running
   RTC.start();          // Fai partire il clock
}

void SetOrologio()
{
  setTime(RTC.get(DS1307_HR,true),RTC.get(DS1307_MIN,false),RTC.get(DS1307_SEC,false),RTC.get(DS1307_DATE,false),RTC.get(DS1307_MTH,false),RTC.get(DS1307_YR,false)-2000);
}

void TxMsg(char buffer[15])
{
   vw_send((uint8_t *)buffer, strlen(buffer));
   vw_wait_tx(); // Wait until the whole message is gone
   buffer[0]='\0';
   Serial.println("Comando Inviato");
}

// Forward declaration of the toplevel element
M2_EXTERN_HLIST(top_el_expandable_menu);

/* SetDate 1-1  */
uint8_t dt_day = 1;
uint8_t dt_month = 1;
uint8_t dt_year = 12;
uint8_t dt_min = 1;
uint8_t dt_hour = 1;

void dt_current(void)
{
  Serial.print(RTC.get(DS1307_HR,true)); //read the hour and also update all the values by pushing in true
  Serial.print(":");
  Serial.print(RTC.get(DS1307_MIN,false));//read minutes without update (false)
  Serial.print(":");
  Serial.print(RTC.get(DS1307_SEC,false));//read seconds
  Serial.print("      ");                 // some space for a more happy life
  Serial.print(RTC.get(DS1307_DATE,false));//read date
  Serial.print("/");
  Serial.print(RTC.get(DS1307_MTH,false));//read month
  Serial.print("/");
  Serial.print(RTC.get(DS1307_YR,false)); //read year
  Serial.println();
}


void dt_get_from_RTC(void)
{
  dt_day = RTC.get(DS1307_DATE,false);
  dt_month = RTC.get(DS1307_MTH,false);
  dt_year = (RTC.get(DS1307_YR,false)-2000);
  dt_min = RTC.get(DS1307_MIN,false);
  dt_hour = RTC.get(DS1307_HR,true); 
}

void dt_put_to_RTC(void)
{
  RTC.stop();
  RTC.set(DS1307_DATE,dt_day);
  RTC.set(DS1307_MTH,dt_month);
  RTC.set(DS1307_YR,dt_year);
  RTC.start(); 
  
}

void time_put_to_RTC(void)
{
  RTC.stop();
  RTC.set(DS1307_HR,dt_hour);
  RTC.set(DS1307_MIN,dt_min);
  RTC.start(); 
  SetOrologio();
}
void dt_ok_date(m2_el_fnarg_p fnarg)  
{
  dt_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void time_ok_button(m2_el_fnarg_p fnarg)
{
  time_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void view_ok_button(m2_el_fnarg_p fnarg)
{
  dt_current();
  m2_SetRoot(&top_el_expandable_menu);  
}

M2_U8NUM(el_dt_year, "c2", 0,99,&dt_year);
M2_LABEL(el_dt_sep1, "b1", "-");
M2_U8NUM(el_dt_month, "c2", 1,12,&dt_month);
M2_LABEL(el_dt_sep2, "b1", "-");
M2_U8NUM(el_dt_day, "c2", 1,31,&dt_day);

M2_LIST(list_date) = { &el_dt_year, &el_dt_sep1, &el_dt_month, &el_dt_sep2, &el_dt_day };
M2_HLIST(el_date, NULL, list_date);

M2_ROOT(el_dt_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_dt_ok, NULL, "ok", dt_ok_date);
M2_LIST(list_dt_buttons) = {&el_dt_cancel, &el_dt_ok };
M2_HLIST(el_dt_buttons, NULL, list_dt_buttons);

M2_LIST(list_dt) = {&el_date, &el_dt_buttons };
M2_VLIST(el_top_dt, NULL, list_dt);

/* Set time 1-2 ---------------------------------------------------------------------*/

M2_U8NUM(el_dt_hour, "c2", 0,23,&dt_hour);
M2_LABEL(el_dt_c, "b1", ":");
M2_U8NUM(el_dt_min, "c2", 0,59,&dt_min);

M2_LIST(list_time) = { &el_dt_hour, &el_dt_c, &el_dt_min};
M2_HLIST(el_time, NULL, list_time);

M2_ROOT(el_time_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_time_ok, NULL, "ok", time_ok_button);
M2_LIST(list_time_buttons) = {&el_time_cancel, &el_time_ok };
M2_HLIST(el_time_buttons, NULL, list_time_buttons);

M2_LIST(list_t) = {&el_time, &el_time_buttons};
M2_VLIST(el_top_time, NULL, list_t);

/* View DateTime 1-3  */
M2_LABEL(el_dt_view, "b1", "DateTimeNow");
M2_LIST(list_view) = {&el_dt_view};
M2_HLIST(el_view, NULL, list_view);

M2_ROOT(el_view_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_view_ok, NULL, "ok", view_ok_button);
M2_LIST(list_view_buttons) = {&el_view_cancel, &el_view_ok };
M2_HLIST(el_view_buttons, NULL, list_view_buttons);

M2_LIST(list_v) = {&el_view, &el_view_buttons};
M2_VLIST(el_top_view, NULL, list_v);

/* -------------- DEVICE 2 ----------------------*/

uint8_t u8dev = 0;
uint8_t u8stato = 0;

void fn_clean_dev(m2_el_fnarg_p fnarg) {
  u8dev = 0;
}

void dev_ok_button(m2_el_fnarg_p fnarg) {
  
  if (u8dev <= 9) 
    {
     messaggio[1] = u8dev + 48;  
     messaggio[2] = '#';
     messaggio[3] = '\0';
    }
  
    if (u8dev > 9)
    {
      int x,y;
      y = u8dev/10;
      messaggio[1] = y + 48;
      x = u8dev % 10;
      messaggio[2] = x + 48;
      messaggio[3] = '#';
      messaggio[4] = '\0';
    }    
    
    if (u8stato == 0)
    {
      messaggio[0] ='*';
    }    
    
    if (u8stato == 1)
    {
      messaggio[0] ='1';
    }   
       
  TxMsg(messaggio);
  messaggio[0] = '\0';
}

void dev_stato_button(m2_el_fnarg_p fnarg) {
 // procedura per recuperare lo stato della periferica
 
}

M2_LABEL(el_num_label1, NULL, "Select Dev:");
M2_U8NUM(el_num_1, NULL, 0, 64, &u8dev);

M2_LABEL(el_num_label2,NULL, "Turn Off/On:");
M2_U8NUM(el_num_2, NULL, 0, 1, &u8stato);

M2_BUTTON(el_num_zero, "f4", "zero", fn_clean_dev);
M2_BUTTON(el_dev_ok, NULL, "ok", dev_ok_button);
M2_BUTTON(el_dev_stato, NULL, "stato", dev_stato_button);
M2_ROOT(el_num_goto_top, "f4", "back", &top_el_expandable_menu);

M2_LIST(num_list) = { 
    &el_num_label1, &el_num_1, 
    &el_num_label2, &el_num_2, 
    &el_dev_ok, &el_num_zero, &el_num_goto_top,&el_dev_stato
};
M2_GRIDLIST(el_num_menu, "c2", num_list);


/* main menu */
m2_menu_entry m2_2lmenu_data[] = 
{
  { "DATE/TIME 1", NULL },
  { ". SetDate 1-1", &el_top_dt },
  { ". SetTime 1-2", &el_top_time},
  { ". View DateTime 1-3", &el_top_view},
  { "DEVICES 2", NULL},
  { ". Manage 2-1", &el_num_menu},
  { NULL, NULL },
};

uint8_t m2_2lmenu_first;
uint8_t m2_2lmenu_cnt;

M2_2LMENU(el_2lmenu,"l4e1w12",&m2_2lmenu_first,&m2_2lmenu_cnt, m2_2lmenu_data,'+','-','\0');
M2_VSB(el_vsb, "l4w1r1", &m2_2lmenu_first, &m2_2lmenu_cnt);
M2_LIST(list_2lmenu) = { &el_2lmenu, &el_vsb };
M2_HLIST(top_el_expandable_menu, NULL, list_2lmenu);

// m2 object and constructor
M2tk m2(&top_el_expandable_menu, m2_es_arduino_serial, m2_eh_2bs, m2_gh_arduino_serial);

void setup() 
{
  
  LeggiEeprom();
  Comunicazione();
  
  
  
}

void loop() {
  Timer();
  m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey();
  Timer();
}

Thanks for the support,
gnux

After a quick review of your code i do not yet see a big mistake.

Is the Menu working without EEPROM?
Maybe there is a RAM overflow (too much of RAM consumption).

Oliver

Hi Oliver,
without EEPROM the menu is working ... so it's working also with EEPROM,
but only the timer doesn't works ... so now What I've in mind to try is that:

a) check timer alone --> it's working
b) check the timer with DS1307 and see if it working
c) check timer with DS1307 + a little bit menu and see it's working
d) check timer with EEPROM+DS1307+MENU and see it's working

make sense for you ?

Thanks
Andrea

Hi Oliver ...

I've tried steps by step ... this result ...

a) check timer alone --> it's working --> Ok
b) check the timer with DS1307 and see if it working --> Ok
c) check the timer with eeprom+DS1307 --> Ok
d) check timer with DS1307 + a little bit menu and see it's working -->WITHOUT EEPROM--> not ok
e) check timer with DS1307 + a little bit menu WITH EEPROM --> not ok

So I've tried to move the timer from loop to setup and from setup to loop without any success ....

this below is an example without no more complicated ... maybe you have one sketch that it's working with timer ?

to be honest at this point and don't know what check ... seems that the timer routine is never call ...

/*
 * TimeAlarmExample.pde
 *
 * This example calls alarm functions at 8:30 am and at 5:45 pm (17:45)
 * and simulates turning lights on at night and off in the morning
 * A weekly timer is set for Saturdays at 8:30:30
 *
 * A timer is called every 15 seconds
 * Another timer is called once only after 10 seconds
 *
 * At startup the time is set to Jan 1 2011  8:29 am
 */
#include <Wire.h> 
#include <DS1307.h> 
#include <Time.h>
#include <TimeAlarms.h>
#include <LiquidCrystal.h>	// ensure that the include path is set
#include "M2tk.h"

M2_EXTERN_HLIST(top_el_expandable_menu);


/*======================================================================*/
/* number entry */

uint8_t u8num = 0;
uint32_t u32num = 0;

void fn_num_zero(m2_el_fnarg_p fnarg) {
  u8num = 0;
  u32num = 0;
}

M2_LABEL(el_num_label1, NULL, "U8:");
M2_U8NUM(el_num_1, NULL, 0, 255, &u8num);

M2_LABEL(el_num_label2, NULL, "U32:");
M2_U32NUM(el_num_2, "c5", &u32num);

M2_BUTTON(el_num_zero, "f4", " zero ", fn_num_zero);
M2_ROOT(el_num_goto_top, "f4", " back ", &top_el_expandable_menu);

M2_LIST(num_list) = { 
    &el_num_label1, &el_num_1, 
    &el_num_label2, &el_num_2,  
    &el_num_zero, &el_num_goto_top
};
M2_GRIDLIST(el_num_menu, "c2", num_list);

/*=========================================================================*/
/* edit date dialog */

uint8_t dt_day = 1;
uint8_t dt_month = 1;
uint8_t dt_year = 12;

void dt_ok_fn(m2_el_fnarg_p fnarg)  {
  m2_SetRoot(&top_el_expandable_menu);
}

M2_U8NUM(el_dt_year, "c2", 0,99,&dt_year);
M2_LABEL(el_dt_sep1, "b1", "-");
M2_U8NUM(el_dt_month, "c2", 1,12,&dt_month);
M2_LABEL(el_dt_sep2, "b1", "-");
M2_U8NUM(el_dt_day, "c2", 1,31,&dt_day);

M2_LIST(list_date) = { &el_dt_year, &el_dt_sep1, &el_dt_month, &el_dt_sep2, &el_dt_day };
M2_HLIST(el_date, NULL, list_date);

M2_ROOT(el_dt_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_dt_ok, NULL, "ok", dt_ok_fn);
M2_LIST(list_dt_buttons) = {&el_dt_cancel, &el_dt_ok };
M2_HLIST(el_dt_buttons, NULL, list_dt_buttons);

M2_LIST(list_dt) = {&el_date, &el_dt_buttons };
M2_VLIST(el_top_dt, NULL, list_dt);

/*=========================================================================*/
/* radio */

uint8_t select_color = 0;

void fn_radio_ok(m2_el_fnarg_p fnarg) {
  /* accept selection */
  m2_SetRoot(&top_el_expandable_menu);
}

void fn_radio_cancel(m2_el_fnarg_p fnarg) {
  /* discard selection */
  m2_SetRoot(&top_el_expandable_menu);
}

M2_LABEL(el_radio_label1, NULL, "red");
M2_RADIO(el_radio_radio1, "v0", &select_color);

M2_LABEL(el_radio_label2, NULL, "green");
M2_RADIO(el_radio_radio2, "v1", &select_color);

M2_LABEL(el_radio_label3, NULL, "blue");
M2_RADIO(el_radio_radio3, "v2", &select_color);

M2_BUTTON(el_radio_cancel, NULL, "cancel", fn_radio_cancel);
M2_BUTTON(el_radio_ok, NULL, "ok", fn_radio_ok);

M2_LIST(list_radio) = { 
    &el_radio_label1, &el_radio_radio1, 
    &el_radio_label2, &el_radio_radio2,  
    &el_radio_label3, &el_radio_radio3, 
    &el_radio_cancel, &el_radio_ok 
};
M2_GRIDLIST(top_el_radio, "c2",list_radio);

/*=========================================================================*/
/* combo */

uint8_t select_priority = 0;

void fn_combo_ok(m2_el_fnarg_p fnarg) {
  /* accept selection */
  m2_SetRoot(&top_el_expandable_menu);
}

void fn_combo_cancel(m2_el_fnarg_p fnarg) {
  /* discard selection */
  m2_SetRoot(&top_el_expandable_menu);
}

const char *fn_idx_to_color(uint8_t idx)
{
  if ( idx == 0 )
    return "red";
  else if (idx == 1 )
    return "green";
  return "blue";
}

const char *fn_idx_to_priority(uint8_t idx)
{
  switch(idx)
  {
    case 0: return "lowest";
    case 1: return "low";
    case 2: return "medium";
    case 3: return "high";
    case 4: return "highest";
  }
  return "";
}


M2_LABEL(el_combo_label1, NULL, "Color:");
M2_COMBO(el_combo_combo1, NULL, &select_color, 3, fn_idx_to_color);

M2_LABEL(el_combo_label2, NULL, "Prio.: ");
M2_COMBO(el_combo_combo2, "v1", &select_priority, 5, fn_idx_to_priority);

M2_BUTTON(el_combo_cancel, NULL, "cancel", fn_combo_cancel);
M2_BUTTON(el_combo_ok, NULL, " ok ", fn_combo_ok);

M2_LIST(list_combo) = { 
    &el_combo_label1, &el_combo_combo1, 
    &el_combo_label2, &el_combo_combo2,  
    &el_combo_cancel, &el_combo_ok 
};
M2_GRIDLIST(top_el_combo, "c2",list_combo);

/*=========================================================================*/
/* main menu */


// Left entry: Menu name. Submenus must have a '.' at the beginning
// Right entry: Reference to the target dialog box (In this example all menus call the toplevel element again
m2_menu_entry m2_2lmenu_data[] = 
{
  { "Numeric", NULL },
  { ". U8 U32", &el_num_menu },
  { ". Date", &el_top_dt },
  { "Radio", &top_el_radio },
  { "Combo", &top_el_combo },
  { NULL, NULL },
};

// The first visible line and the total number of visible lines.
// Both values are written by M2_2LMENU and read by M2_VSB
uint8_t m2_2lmenu_first;
uint8_t m2_2lmenu_cnt;

// M2_2LMENU definition
// Option l4 = four visible lines
// Option e1 = first column has a width of 1 char
// Option w12 = second column has a width of 12 chars

M2_2LMENU(el_2lmenu,"l4e1w12",&m2_2lmenu_first,&m2_2lmenu_cnt, m2_2lmenu_data,'+','-','\0');
M2_VSB(el_vsb, "l4w1r1", &m2_2lmenu_first, &m2_2lmenu_cnt);
M2_LIST(list_2lmenu) = { &el_2lmenu, &el_vsb };
M2_HLIST(top_el_expandable_menu, NULL, list_2lmenu);

// m2 object and constructor
M2tk m2(&top_el_expandable_menu, m2_es_arduino_serial, m2_eh_4bs, m2_gh_arduino_serial);
TimeElements tm,tm1;

byte memoria[105];

String readString;
#include <EEPROM.h>
void setup()
{
  LeggiEeprom();
  RTC.stop();
  RTC.set(DS1307_DATE,23);
  RTC.set(DS1307_MTH,01);
  RTC.set(DS1307_YR,2013);
  RTC.set(DS1307_HR,1);
  RTC.set(DS1307_MIN,3);
  RTC.start(); 
  int c;
  
  Serial.begin(9600);
  setTime(RTC.get(DS1307_HR,true),RTC.get(DS1307_MIN,false),RTC.get(DS1307_SEC,false),RTC.get(DS1307_DATE,false),RTC.get(DS1307_MTH,false),RTC.get(DS1307_YR,false)-2000);
  // setTime(1,3,00,23,1,13); // set time to Saturday 8:29:00am Jan 1 2011
  
  tm.Year = (memoria[6]+ 2000) - 1970;
  tm.Month = (memoria[5]);
  tm.Day = memoria[4];
  tm.Hour =  memoria[2];
  tm.Minute =  memoria[3];
  tm.Second = 00;
  tm1.Year = (memoria[11]+2000) -1970;
  tm1.Month = (memoria[10]);
  tm1.Day = (memoria[9]);
  tm1.Hour = (memoria[7]);
  tm1.Minute = (memoria[8]);
  tm1.Second = 00;
  
   Alarm.triggerOnce(makeTime(tm), OnceOnly);
  Alarm.triggerOnce(makeTime(tm1), OnceOnly1);
  
  // Alarm.timerOnce(makeTime(tm), OnceOnly) ;
  // Serial.println("Make Time");
  
  // Alarm.timerRepeat(15, Repeats);            // timer for every 15 seconds    
               // called once after 10 seconds 
}

void  loop(){  
 
   m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey();
 
  
  // digitalClockDisplay();
  
  // Alarm.delay(1000); // wait one second between clock display
  
  
}

// functions to be called when an alarm triggers:

void OnceOnly(){
  Serial.println("Inizio Timer");  
}

void OnceOnly1(){
  Serial.println("Fine Timer");  
}
void digitalClockDisplay()
{
  // digital clock display of the time
  Serial.print(hour());
  printDigits(minute());
  printDigits(second());
  Serial.println(); 
}

void printDigits(int digits)
{
  Serial.print(":");
  if(digits < 10)
    Serial.print('0');
  Serial.print(digits);
}

void LeggiEeprom()
{
  int ind;
   for (ind =0; ind <= 105 ; ind++)
      {
        memoria[ind] = EEPROM.read(ind);
      }   
}

thanks
gnux

Hi Oliver,
after one day that work on that ... I can say this:

  • RTC and Arduino Clock are correctly set
  • Without use a timer but only put a standard clock for example clock that change in working
  • With timer for sure the timer are correct but stop to work ...

it's possible that timer is not compatible with menu or viceversa ?

Thanks for the support,

gnux

Hi

Excause me for not reading this thread more often.

I do not know much about the timer itself. What i can say: M2tklib will use

  • digitalRead
  • Serial interface for the Monitor Simulation
    Can this do any harm to your system?

What you could do is to comment out

  m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey();

and check if things are working again.
Maybe you can also check the timer procedures with some small menu.
If this is working, that it is probably a memory issue.
Which Arduino do you use?

Oliver

No problem Oliver don't worry :slight_smile: thanks for your help and time :slight_smile: I was writing you because during this day I was free time for do the try just it :slight_smile: ...

So due the fact I've note this ... for example without set a timer retriving date and time from RTC it's working ...

void loop() {
  
     
    // Alarm.triggerOnce(makeTime(tm0start), StartTimer0);
     
     
     
     // Alarm.triggerOnce(makeTime(tm0stop), StopTimer0);
  Serial.print(RTC.get(DS1307_HR,true)); //read the hour and also update all the values by pushing in true
  Serial.print(":");
  Serial.print(RTC.get(DS1307_MIN,false));//read minutes without update (false)
  Serial.print(":");
  Serial.print(RTC.get(DS1307_SEC,false));//read seconds
  Serial.print("      ");                 // some space for a more happy life
  Serial.print(RTC.get(DS1307_DATE,false));//read date
  Serial.print("/");
  Serial.print(RTC.get(DS1307_MTH,false));//read month
  Serial.print("/");
  Serial.print(RTC.get(DS1307_YR,false)); //read year
  Serial.println();
  
  m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey();
  
}

Then what i mean it that execute the instruction into the loop also ...

I think that is necessary troubleshoot Time.h library , but i don't know if i will be able to this ...
the library menu use some particular resources sorry if I ask you but could be useful tomorrow when i'll try to compare the 2 library ....

so I'm using arduino 2560 mega ...

Thanks for the big support,
have a good night ...
gnux

Schermata 2013-01-22 a 21.45.08.png

Good Morning Oliver,
so, finally it's working :slight_smile: ... after a lots of try I've understand what was really missing ...

a)

setTime(RTC.get(DS1307_HR,true),RTC.get(DS1307_MIN,false),RTC.get(DS1307_SEC,false),RTC.get(DS1307_DATE,false),RTC.get(DS1307_MTH,false),13)

Looking the code you will note that like Year I've explict "13" --> "2013" before there was this :

setTime(RTC.get(DS1307_HR,true),RTC.get(DS1307_MIN,false),RTC.get(DS1307_SEC,false),RTC.get(DS1307_DATE,false),RTC.get(DS1307_MTH,false),(RTC.get(DS1307_YR,false)-2000))

That I think means that doesn't accept the calcution in this way in fact is necessary put (), but I will do other try just to understand ... Maybe I will calculate into a variable and then I will pass the value to the function ...

The other problems was that, into the loop is necessary put

Alarm.delay(1000);

Otherwise the timer will not start ...

Ok with the timer of course but i think is correct there is a little bit delay because off course in background there is the timer that is working ...

So now i will study an efficient manner how to manage the timer ... I'll let you know because I think that you are a very good teacher... I've to found the way for do a input menu for a timer ... :slight_smile: maybe you can help me with this :slight_smile:

my idea is do a menu for a timer structure in this way:

Timer:

  • set timer
    --> select dev
    --> enable / disable
    --> put time start date time
    --> put time stop date time
    (could be the best do start / stop time with only row if is possible)

  • View Timer Set

so for select a dev and enable/disable i think is not a problem ... for the other could be a issues ...
because im not able to manage at the top the menu :frowning:

Thanks and have nice day,
gnux

Hi

Maybe i have not yet fully understood your problem. Do you want to program exactly one timer or more than one timer?

In either case, you need to know, what exactly needs to be entered for one timer.
You mention date and time, start and dev.
But it probably needs to be cut down to atomic data times like byte and words.
So this is probably the first step: To break down the required information to small pieces.
For example in this way:
time: hour (byte, 0..23), minute (byte, 0..59), seconds (0..59)
start: could be represented as a status byte: 0 for timer stopped, 1 for time is running. In that sense the variable should be better called "is_running". By the way it is a good practice to have a naming convention for variables. For example if all boolean variables start with "is_" then they are easily seen in the code. You could also do something like
if ( is_running ) ...
which will be executed if the time "runs"
An open question to me is the mentioned "dev" (select dev). What exactly needs to be selected? How much selectable items are there. Can it be mapped to some integer (byte) variable?

So, again, first step is to break down all this to variables. Then write some nice dialog entries for it.

Once done, the question is: Is there only one timer or are there more timers to handle. Of course the existing dialogs should be reused for each different set of timer configuration variables.

if you need to handle more than one timer, i think it will make sense to study tutorial 8: Google Code Archive - Long-term storage for Google Code Project Hosting.. Tutorial 8 has a configuration menu for the PWM output. Compare your timer data with the PWM output in this tutorial. This tutorial also introduces a top level menu. And: The PWM example is also available in the Arduino IDE (if m2tklib is installed correctly)

Oliver

If Alarm.delay(1000) is causing you a problem because it blocks your code for 1 second then you should consider reducing the delay time or better, only calling Alarm.delay() when it will not interfere with the operation of your program, or both, of course.

I think should be better reduce the delay maybe ...

because otherwise how can the timer start ?

Thanks
gnux

Hi Oliver,
you have exactly center my ideas ... :slight_smile:

So:

  • N°Dev --> are the number of the devices
    for example: I can set up timer for the devices from 0 to 7 ...

when I setup the timer from a menu do this:

  • pass the menu value to:
    a) a timer
    b) in eeprom
    c) array (copy of eeprom)
    in this way if the electricity will be down automatically a timer will be set again if wrote into the eeprom.
    The first thing that the software do is read eeprom and pass the value to a local array that is identical to the eeprom.

Then my ideas is change dynamically a timer from a menu ... make sense for you ?

thanks gnux

Hi Oliver,
sorry if i ask you ... But I need to more information about the menu ... looking this menu:

#include <LiquidCrystal.h>	// ensure that the include path is set
#include "M2tk.h"
#if defined(ARDUINO) && ARDUINO >= 100
#include "Arduino.h"
#else
#include "WProgram.h"
#endif

// Forward declaration of the toplevel element
M2_EXTERN_HLIST(top_el_expandable_menu);

/* SetDate 1-1  */
uint8_t dt_day = 1;
uint8_t dt_month = 1;
uint8_t dt_year = 12;
uint8_t dt_min = 1;
uint8_t dt_hour = 1;

void dt_current(void)
{
  
}


void dt_get_from_RTC(void)
{
  
}

void dt_put_to_RTC(void)
{
 
}

void dt_ok_date(m2_el_fnarg_p fnarg)  
{
  dt_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void time_ok_button(m2_el_fnarg_p fnarg)
{
 dt_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void view_ok_button(m2_el_fnarg_p fnarg)
{
  dt_current();
  m2_SetRoot(&top_el_expandable_menu);  
}

M2_U8NUM(el_dt_year, "c2", 0,99,&dt_year);
M2_LABEL(el_dt_sep1, "b1", "-");
M2_U8NUM(el_dt_month, "c2", 1,12,&dt_month);
M2_LABEL(el_dt_sep2, "b1", "-");
M2_U8NUM(el_dt_day, "c2", 1,31,&dt_day);

M2_LIST(list_date) = { &el_dt_year, &el_dt_sep1, &el_dt_month, &el_dt_sep2, &el_dt_day };
M2_HLIST(el_date, NULL, list_date);

M2_ROOT(el_dt_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_dt_ok, NULL, "ok", dt_ok_date);
M2_LIST(list_dt_buttons) = {&el_dt_cancel, &el_dt_ok };
M2_HLIST(el_dt_buttons, NULL, list_dt_buttons);

M2_LIST(list_dt) = {&el_date, &el_dt_buttons };
M2_VLIST(el_top_dt, NULL, list_dt);

/* Set time 1-2 ---------------------------------------------------------------------*/

M2_U8NUM(el_dt_hour, "c2", 0,23,&dt_hour);
M2_LABEL(el_dt_c, "b1", ":");
M2_U8NUM(el_dt_min, "c2", 0,59,&dt_min);

M2_LIST(list_time) = { &el_dt_hour, &el_dt_c, &el_dt_min};
M2_HLIST(el_time, NULL, list_time);

M2_ROOT(el_time_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_time_ok, NULL, "ok", time_ok_button);
M2_LIST(list_time_buttons) = {&el_time_cancel, &el_time_ok };
M2_HLIST(el_time_buttons, NULL, list_time_buttons);

M2_LIST(list_t) = {&el_time, &el_time_buttons};
M2_VLIST(el_top_time, NULL, list_t);

/* View DateTime 1-3  */
M2_LABEL(el_dt_view, "b1", "DateTimeNow");
M2_LIST(list_view) = {&el_dt_view};
M2_HLIST(el_view, NULL, list_view);

M2_ROOT(el_view_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_view_ok, NULL, "ok", view_ok_button);
M2_LIST(list_view_buttons) = {&el_view_cancel, &el_view_ok };
M2_HLIST(el_view_buttons, NULL, list_view_buttons);

M2_LIST(list_v) = {&el_view, &el_view_buttons};
M2_VLIST(el_top_view, NULL, list_v);

/* -------------- DEVICE 2 ----------------------*/

uint8_t u8dev = 0;
uint8_t u8stato = 0;

void fn_clean_dev(m2_el_fnarg_p fnarg) {
  u8dev = 0;
}

void dev_ok_button(m2_el_fnarg_p fnarg) {
  
}

void dev_stato_button(m2_el_fnarg_p fnarg) {
 // procedura per recuperare lo stato della periferica
 
}

M2_LABEL(el_num_label1, NULL, "Select Dev:");
M2_U8NUM(el_num_1, NULL, 0, 64, &u8dev);

M2_LABEL(el_num_label2,NULL, "Turn Off/On:");
M2_U8NUM(el_num_2, NULL, 0, 1, &u8stato);

M2_BUTTON(el_num_zero, "f4", "zero", fn_clean_dev);
M2_BUTTON(el_dev_ok, NULL, "ok", dev_ok_button);
M2_BUTTON(el_dev_stato, NULL, "stato", dev_stato_button);
M2_ROOT(el_num_goto_top, "f4", "back", &top_el_expandable_menu);

M2_LIST(num_list) = { 
    &el_num_label1, &el_num_1, 
    &el_num_label2, &el_num_2, 
    &el_dev_ok, &el_num_zero, &el_num_goto_top,&el_dev_stato
};
M2_GRIDLIST(el_num_menu, "c2", num_list);

/* --------------------------- TIMER 3-------------------------- */

uint8_t u8devTimer = 0;  // used for select the devices
uint8_t IsTimerEnable = 0;     // used for say if enable or disable 0 disable 1 enable
// set start date time
uint8_t dt_day_start = 1;
uint8_t dt_month_start = 1;
uint8_t dt_year_start = 13;
uint8_t dt_min_start = 1;
uint8_t dt_hour_start = 1;
// set stop date time
uint8_t dt_day_stop = 1;
uint8_t dt_month_stop = 1;
uint8_t dt_year_stop = 13;
uint8_t dt_min_stop = 1;
uint8_t dt_hour_stop = 1;

void fn_clean_dev_timer(m2_el_fnarg_p fnarg) {
  u8devTimer = 0;
}

void dev_ok_button_timer(m2_el_fnarg_p fnarg) {
  m2_SetRoot(&top_el_expandable_menu);
}

M2_LABEL(el_num_timer, NULL, "Select Dev:");
M2_U8NUM(el_num_t1, NULL, 0, 64, &u8devTimer);

M2_LABEL(el_num_timer2,NULL, "Enable/Disable");
M2_U8NUM(el_num_t2, NULL, 0, 1, &IsTimerEnable);
// Time
M2_LABEL(el_num_timer3,NULL, "Start When");
M2_U8NUM(el_dt_hour_t1start, "c2", 0,23,&dt_hour_start);
M2_LABEL(el_dt_ts, "b1", ":");
M2_U8NUM(el_dt_min_t1start, "c2", 0,59,&dt_min_start);
// Date
M2_U8NUM(dt_year_t1start, "c2", 0,99,&dt_year_start);
M2_LABEL(el_dt_ts1, "b1", "-");
M2_U8NUM(dt_month_t1start, "c2", 1,12,&dt_month_start);
M2_LABEL(el_dt_ts2, "b1", "-");
M2_U8NUM(dt_day_t1start, "c2", 1,31,&dt_day_start);

M2_BUTTON(el_num_zero_timer, "f4", "zero", fn_clean_dev_timer);
M2_BUTTON(el_dev_ok_timer, NULL, "ok", dev_ok_button_timer);
// M2_BUTTON(el_dev_stato_timer, NULL, "stato", dev_stato_button_timer);
M2_ROOT(el_num_goto_top_timer, "f4", "back", &top_el_expandable_menu);

M2_LIST(num_list_timer) = { 
    &el_num_timer, &el_num_t1, 
    &el_num_timer2, &el_num_t2,
    &el_num_timer3, &el_dt_hour_t1start,&el_dt_ts,&el_dt_min_t1start, 
    &el_dev_ok_timer, &el_num_zero_timer, &el_num_goto_top_timer
};
M2_GRIDLIST(el_num_menu_timer, "c2", num_list_timer);


/* --------------------------- main menu-------------------------- */
m2_menu_entry m2_2lmenu_data[] = 
{
  { "DATE/TIME 1", NULL },
  { ". SetDate 1-1", &el_top_dt },
  { ". SetTime 1-2", &el_top_time},
  { ". View DateTime 1-3", &el_top_view},
  { "DEVICES 2", NULL},
  { ". Manage 2-1", &el_num_menu},
  { "TIMER 3", NULL},
  { ". SetTimer 3-1", &el_num_menu_timer},
  { ". ViewTimer 3-2", &el_top_view},
  { NULL, NULL },
};

uint8_t m2_2lmenu_first;
uint8_t m2_2lmenu_cnt;

M2_2LMENU(el_2lmenu,"l4e1w12",&m2_2lmenu_first,&m2_2lmenu_cnt, m2_2lmenu_data,'+','-','\0');
M2_VSB(el_vsb, "l4w1r1", &m2_2lmenu_first, &m2_2lmenu_cnt);
M2_LIST(list_2lmenu) = { &el_2lmenu, &el_vsb };
M2_HLIST(top_el_expandable_menu, NULL, list_2lmenu);



// m2 object and constructor
M2tk m2(&top_el_expandable_menu, m2_es_arduino_serial, m2_eh_2bs, m2_gh_arduino_serial);

/* --------------------------- end menu-------------------------- */

void setup() 
{
     
}

void loop() {
  
     
  // Timer();
  m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey();
}

right now I have this out put for the time if you try the sketch:

+DATE/TIME 1
+DEVICES 2
-TIMER 3
[ SetTimer 3-1
s
Start 01
: 01
ok zero
back

But I would like to have the outoput for the timer in this way:

+DATE/TIME 1
+DEVICES 2
-TIMER 3
[ SetTimer 3-1
s
Sel Dev: 01 E/D: 0
Start: 01:01 + date (need to implement again because i want to do step by step)
Stop: 01:01 + date (need to implement again because i want to do step by step)
Ok Zero Back

I think is related to the hierarchy mention by you :slight_smile: but i need to take more feeling with that :slight_smile:

is possible from your point of view ?

Thanks for the support and have nice week end :slight_smile:

Hi

I think there is some missunderstanding of the GRIDLIST. I would suggest to read documentation about it. Maybe you could instead of the GRIDLIST use the XYLIST, which might be simpler to use. With the XYLIST you must provide the left position of each child element, which lets you create more easily the menu element.

Oliver

gnusso:
I can set up timer for the devices from 0 to 7 ...

TimeAlarms only normally allows up to 6 alarms to be set concurrently. You can change a line in TimeAlarms.h to increase the limit.

#define dtNBR_ALARMS 6   // max is 255

Thanks to you guys :slight_smile: I will read the documentation for the menu :slight_smile: and I will try your suggest ...

So thanks also for the allarm is important Know how many alarms is possible set ...

warms regards,
gnux

Hi Oliver,
today after hours of I've build the menu ... I've understood how to use x,y list ... but now there is a little problem ... so I've available a lcd 20x4 ... well with my sketch i need more over di 4 rows ... with the sketch that I've did I've note that if i add more of 4 columns the display start to have some issues ... below reported the code ... so from my understating the code is correct: the problem is that with 20 character available, what I've understand is that i cannot put time and date only one rows ...because using x,y in cannot format the field ...then If not wrong this is the calculation :

start date: [000][000][000][000][000] --> totally characters used: 27 (without label and separator) then i need to split on more rows ... :frowning: do you know If id possible fix it ?

if try to copy and paste the code you should understand perfectly what I mean :frowning: in attachment screenshoot also ... thansk for the support ... now i start to understand better also the tutorial is more clear now ...
see in the timer menu :wink:

Thanks for the support :slight_smile:

#include <LiquidCrystal.h>	// ensure that the include path is set
#include "M2tk.h"
#if defined(ARDUINO) && ARDUINO >= 100
#include "Arduino.h"
#else
#include "WProgram.h"
#endif

// Forward declaration of the toplevel element
M2_EXTERN_HLIST(top_el_expandable_menu);

/* SetDate 1-1  */
uint8_t dt_day = 1;
uint8_t dt_month = 1;
uint8_t dt_year = 12;
uint8_t dt_min = 1;
uint8_t dt_hour = 1;

void dt_current(void)
{ 
}

void dt_get_from_RTC(void)
{  
}

void dt_put_to_RTC(void)
{
}

void dt_ok_date(m2_el_fnarg_p fnarg)  
{
  dt_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void time_ok_button(m2_el_fnarg_p fnarg)
{
 dt_put_to_RTC();
  m2_SetRoot(&top_el_expandable_menu); 
}

void view_ok_button(m2_el_fnarg_p fnarg)
{
  dt_current();
  m2_SetRoot(&top_el_expandable_menu);  
}

M2_U8NUM(el_dt_year, "c2", 0,99,&dt_year);
M2_LABEL(el_dt_sep1, "b1", "-");
M2_U8NUM(el_dt_month, "c2", 1,12,&dt_month);
M2_LABEL(el_dt_sep2, "b1", "-");
M2_U8NUM(el_dt_day, "c2", 1,31,&dt_day);

M2_LIST(list_date) = { &el_dt_year, &el_dt_sep1, &el_dt_month, &el_dt_sep2, &el_dt_day };
M2_HLIST(el_date, NULL, list_date);

M2_ROOT(el_dt_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_dt_ok, NULL, "ok", dt_ok_date);
M2_LIST(list_dt_buttons) = {&el_dt_cancel, &el_dt_ok };
M2_HLIST(el_dt_buttons, NULL, list_dt_buttons);

M2_LIST(list_dt) = {&el_date, &el_dt_buttons };
M2_VLIST(el_top_dt, NULL, list_dt);

/* Set time 1-2 ---------------------------------------------------------------------*/

M2_U8NUM(el_dt_hour, "c2", 0,23,&dt_hour);
M2_LABEL(el_dt_c, "b1", ":");
M2_U8NUM(el_dt_min, "c2", 0,59,&dt_min);

M2_LIST(list_time) = { &el_dt_hour, &el_dt_c, &el_dt_min};
M2_HLIST(el_time, NULL, list_time);

M2_ROOT(el_time_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_time_ok, NULL, "ok", time_ok_button);
M2_LIST(list_time_buttons) = {&el_time_cancel, &el_time_ok };
M2_HLIST(el_time_buttons, NULL, list_time_buttons);

M2_LIST(list_t) = {&el_time, &el_time_buttons};
M2_VLIST(el_top_time, NULL, list_t);

/* View DateTime 1-3  */
// M2_LABEL(el_dt_view, "b1", "DateTimeNow");
//M2_LIST(list_view) = {&el_dt_view};
//M2_HLIST(el_view, NULL, list_view);

// M2_ROOT(el_view_cancel, NULL, "cancel", &top_el_expandable_menu);
M2_BUTTON(el_view_ok, NULL, "ok", view_ok_button);
M2_LIST(list_view_buttons) = {&el_view_ok };
M2_HLIST(el_view_buttons, NULL, list_view_buttons);

M2_LIST(list_v) = {&el_view_buttons};
M2_VLIST(el_top_view, NULL, list_v);

/* -------------- DEVICE 2 ----------------------*/

uint8_t u8dev = 0;
uint8_t u8stato = 0;

void fn_clean_dev(m2_el_fnarg_p fnarg) {
  u8dev = 0;
}

void dev_ok_button(m2_el_fnarg_p fnarg) {
  m2_SetRoot(&top_el_expandable_menu);
}

void dev_stato_button(m2_el_fnarg_p fnarg) {
 // procedura per recuperare lo stato della periferica
 
}

M2_LABEL(el_num_label1, NULL, "Select Dev:");
M2_U8NUM(el_num_1, NULL, 0, 64, &u8dev);

M2_LABEL(el_num_label2,NULL, "Turn Off/On:");
M2_U8NUM(el_num_2, NULL, 0, 1, &u8stato);

M2_BUTTON(el_num_zero, NULL, "Zero", fn_clean_dev);
M2_BUTTON(el_dev_ok, NULL, "Ok", dev_ok_button);
M2_BUTTON(el_dev_stato, NULL, "State", dev_stato_button);
//M2_ROOT(el_num_goto_top, "f4", "back", &top_el_expandable_menu);

M2_LIST(num_list) = { 
    &el_num_label1, &el_num_1, 
    &el_num_label2, &el_num_2, 
    &el_num_zero, &el_dev_stato, &el_dev_ok 
};
M2_GRIDLIST(el_num_menu, "c2", num_list);

/* --------------------------- TIMER 3-------------------------- */

uint8_t u8devTimer = 0;  // used for select the devices
uint8_t IsTimerEnable = 0;     // used for say if enable or disable 0 disable 1 enable
// set start date time
uint8_t dt_day_start = 1;
uint8_t dt_month_start = 1;
uint8_t dt_year_start = 13;
uint8_t dt_min_start = 1;
uint8_t dt_hour_start = 1;
// set stop date time
uint8_t dt_day_stop = 1;
uint8_t dt_month_stop = 1;
uint8_t dt_year_stop = 13;
uint8_t dt_min_stop = 1;
uint8_t dt_hour_stop = 1;

void fn_clean_dev_timer(m2_el_fnarg_p fnarg) {
  u8devTimer = 0;
}

void dev_ok_button_timer(m2_el_fnarg_p fnarg) {
  m2_SetRoot(&top_el_expandable_menu);
}

M2_LABEL(el_num_timer1, "x0y5", "Dev:");
M2_U8NUM(el_num_t1, "x4y5",0, 64, &u8devTimer);
M2_LABEL(el_num_timer2,"x11y5", "E/D:");
M2_U8NUM(el_num_t2, "x14y5", 0, 1, &IsTimerEnable);

M2_LABEL(el_num_timer3,"x0y4", "Start Time:");
M2_U8NUM(el_dt_hour_t1start,"x10y4", 0,23,&dt_hour_start);
M2_U8NUM(el_dt_min_t1start, "x14y4", 0,59,&dt_min_start);
// DATE
M2_LABEL(el_num_timer4,"x0y3", "S.D.:");
M2_U8NUM(dt_day_t1start, "x5y3", 0,99,&dt_day_start);
M2_U8NUM(dt_month_t1start, "x9y3", 1,12,&dt_month_start);
M2_U8NUM(dt_year_t1start, "x13y3", 1,12,&dt_year_start);

// Stop Timer
M2_LABEL(el_num_timer5,"x0y2", "Stop Time:");
M2_U8NUM(el_dt_hour_t1stop,"x10y2", 0,23,&dt_hour_stop);
M2_U8NUM(el_dt_min_t1stop, "x14y2", 0,59,&dt_min_stop);

M2_LABEL(el_num_timer6,"x0y1", "E.D.:");
M2_U8NUM(dt_day_t1stop, "x5y1", 0,99,&dt_day_stop);
M2_U8NUM(dt_month_t1stop, "x9y1", 1,12,&dt_month_stop);
M2_U8NUM(dt_year_t1stop, "x13y1", 1,12,&dt_year_stop);

// pulsanti
M2_BUTTON(el_num_zero_timer, "x0y0", "Zero", fn_clean_dev_timer);
M2_BUTTON(el_dev_ok_timer, "x8y0", "ok", dev_ok_button_timer);
M2_ROOT(el_num_goto_top_timer, "x13y0", "back", &top_el_expandable_menu);

M2_LIST(num_list_timer) = { &el_num_timer1, &el_num_t1,
                            &el_num_timer2,&el_num_t2,
                            &el_num_timer3,&el_dt_hour_t1start,&el_dt_min_t1start,
                            &el_num_timer4,&dt_day_t1start,&dt_month_t1start,&dt_year_t1start,
                            &el_num_timer5,&el_dt_hour_t1stop, &el_dt_min_t1stop,
                            &el_num_timer6,&dt_day_t1stop,&dt_month_t1stop,&dt_year_t1stop,
                            &el_num_zero_timer,&el_dev_ok_timer,&el_num_goto_top_timer,
                          };
M2_XYLIST(el_num_menu_timer, NULL,num_list_timer);
/* --------------------------- main menu-------------------------- */
m2_menu_entry m2_2lmenu_data[] = 
{
  { "DATE/TIME 1", NULL },
  { ". SetDate 1-1", &el_top_dt },
  { ". SetTime 1-2", &el_top_time},
  { ". View DateTime 1-3", &el_top_view},
  { "DEVICES 2", NULL},
  { ". Manage 2-1", &el_num_menu},
  { "TIMER 3", NULL},
  { ". SetTimer 3-1", &el_num_menu_timer},
  { ". ViewTimer 3-2", &el_top_view},
  { NULL, NULL },
};
uint8_t m2_2lmenu_first;
uint8_t m2_2lmenu_cnt;

M2_2LMENU(el_2lmenu,"l4e1w12",&m2_2lmenu_first,&m2_2lmenu_cnt, m2_2lmenu_data,'+','-','\0');
M2_VSB(el_vsb, "l4w1r1", &m2_2lmenu_first, &m2_2lmenu_cnt);
M2_LIST(list_2lmenu) = { &el_2lmenu, &el_vsb };
M2_HLIST(top_el_expandable_menu, NULL, list_2lmenu);

// m2 object and constructor
M2tk m2(&top_el_expandable_menu, m2_es_arduino_serial, m2_eh_2bs, m2_gh_arduino_serial);
/* --------------------------- end menu-------------------------- */
void setup() 
{    
}

void loop() {
  // Timer();
  m2.checkKey();
  m2.checkKey();
  if ( m2.handleKey() )
    m2.draw();
  m2.checkKey(); 
}

Schermata 2013-01-27 a 09.41.57.png

Hi
All number fields have a format option to limit the number of digits which are displayed.

See here: Google Code Archive - Long-term storage for Google Code Project Hosting.

The relevant part is this:
Format Options
c: Number of digits, e.g. "c2" allows numbers between "00" and "99". Note, that this setting must fit to the min and max value.

This means you get the following results

M2_U8NUM(el,"c3",0,255,&number) --> "[000]"
M2_U8NUM(el,"c2",0,31,&number) --> "[00]"
M2_U8NUM(el,"c1",0,9,&number) --> "[0]"

The "c" format option is the key to format the output.
For your xy-grid you need to combine everything:
M2_U8NUM(el,"x0y1c2",0,31,&number) --> "[00]" at position (0,1)

Hope this helps,
Oliver

Edit: Added "code" section

Hi Oliver :-), thanks for the useful information ,

i was see the tutorial :slight_smile: but and I was guess that was possible combine the value :slight_smile:

but I was pass the value into the wrong way :slight_smile: thanks in this way I think I can stay with the four row :slight_smile:

The concept to mix the format is always good ? thanks gnux :wink:

Hi Oliver,
so now the output is like this :

Dev:[000] E/D:000
01 01 01 01 13
01 01 01 01 13
Zero ok back

I think is not Bad ... what do you think ? How do you will improve this ?

Gnux