Reducing program size for encoder simulator

I made a program to simulate a crankshaft (60-2) and camshaft (arbitrary profile, 1/2 speed of crankshaft) position sensor of an engine. It takes up too much global variable space, plenty of sketch space available though. The general strategy is to have a for loop that goes from 0:719 degrees and looks at two arrays that are either 0 or 1 that I copy paste from excel/.csv file. It works if I give the camshaft a different arbitrary profile and reduce the loop to 360 degrees (0:359) but the resolution isn’t high enough doing it this way to reproduce the 60-2 crankshaft signal.
Any suggestions? I’m an arduino novice and am sure I’m missing something obvious.
Code for the 720 degree code attached, thanks!

ISBcamcrank4_720.ino (3.75 KB)

int ckp[]={1,1,1,0, etc, etc
int cmp[]={0,0,0,0, etc etc

Why are these int when byte would do ?

You could also consider using bits rather than bytes but it would make the program more complicated

For on/off flags, bits are the way to go to save storage space, it's worth investing the time and effort to program them (a necessary technique in the old PDP11/70 programming days!)

I'm sure you will find lots of examples around on the net somewhere. http://playground.arduino.cc/Code/BitMath

Otherwise using Boolean (8 bits instead of 16) should halve your array size i.e. 720 values will take the space of 360 integer values boolean - Arduino Reference

/*
 * Paul Dice pwdice@mtu 9/18/2017
  60-2 crank wheel and ISB6.7 MDT cam sensor 
  


 */
#include <digitalWriteFast.h>
// variables
int rpm=600; //RPM INPUT
int msrev=rpm*.462963; // microseconds per cycle


// the setup function runs once when you press reset or power the board
void setup() {
  // initialize digital pin 13 and 12 as an output.
  pinModeFast(2, OUTPUT);
  pinModeFast(3, OUTPUT);
  pinModeFast(4, OUTPUT);
  Serial.begin(9600);
}
//int ckp[]={1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0};
//int cmp[]={0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,1,1,1,1,1};
// the loop function runs over and over again forever
void loop()
{
  noInterrupts();
 //CKP signal
 for (int i=0; i<=719;i++){
    //if (1==ckp[i]) digitalWriteFast(3, HIGH);
    //if (0==ckp[i]) digitalWriteFast(3, LOW);
    digitalWriteFast((i%6<3?(i<706?1:0):0)); //If i modulo 6 is under 3, and i is less than 706, this is a 1 (ie, HIGH), else a 0. 
    
 //CAM signal
    //if (1==cmp[i]) digitalWriteFast(4, HIGH);

    //if (0==cmp[i]) digitalWriteFast(4, LOW);
    digitalWriteFast(i%45>9); // this is even simpler - this has a 45-entry repeating pattern, and we can use the output of the > operator directly!
 
//TDC pulse for 1 degree.
    if (i==0) digitalWriteFast(2,HIGH);
    delayMicroseconds(msrev);
    if (i==0) digitalWriteFast(2,LOW);
  }
}

How bout this? The values in the array are very predictable, so there’s no need to use a lookup table, just a modulo operator.

If you need to support arbitrary patterns for ckp and cmp, make them byte arrays instead of ints, put them into progmem, and read them with pgm_read_byte(); there are countless examples of this online that do a great job of explaining it, so I won’t belabor this here.

Thanks all, changing int to byte cut the memory usage in half and has me up and running now. When I get a little more time in the next few days I’ll read up on using bits (seems to make more sense than using all the room needed for a byte) as well as progmem. This needs to be fairly easy for someone to change the encoder profiles and offsets so I want to keep the .csv style array declaration.
In the future I will probably need to be able to increase resolution even more so I will be studying on how to make it more efficient. I have some things to look for now so thanks again!