Scrolling on each matrix rather than as a set

[sej7278]

I've tried the AnimateText.ino and the example in README.md and they both do the same thing - the entire text scrolls on each matrix, rather than scrolling across 4 matrices.

So I end up with Demo...Demo...Demo...Demo

rather than D...e...m...o

Any ideas?

Using NodeMCU v1.0 (Amica and Doit.am) and a Wifi-Witty, Arduino 1.6.6 and 1.6.7

[benalb]

¿Have you changed "#define NUMBER_OF_DEVICES 1"?

[ozett]

with me its the same...
the entire text scrolls on each matrix, rather than scrolling across 4 matrices.
how to change? code improvement?

[dotsam]

Turns out this is the same issue I just commented on in #7, where with these boards with 4 modules, they're aligned 90º off, so the scrolling effect doesn't work right. We need something like the "vertical" parameter from this python library https://github.com/rm-hull/max7219

[ozett]

yes, seems you are right.
i did a test with python. it worked there
rm-hull/luma.led_matrix#28

but it would be wonderfull, it it would work with this lib here and the ESP8266.. 😄

[marcelstoer]

Yep, as explained in http://frightanic.com/iot/max7219-library-nodemcu-making/ (my site) you want that rotate option for the 4-in-1 modules and also an invert (upside down) option if you daisy-chain manually.

[adityarao]

Team, if fixed this for 1x4 module mentioned above @ozett and @marcelstoer The modified changes are here: https://github.com/adityarao/MAX7219LedMatrix/

The only change required was to rotate the matrix as below in the LedMatrix::commit() as below:

void LedMatrix::commit() {
    byte index = B0000001;
    byte col; 

    if(deviceOrientation == HORIZONTAL) {
        for (col = 0; col < myNumberOfDevices * 8; col++) {
            sendByte(col / 8, col % 8 + 1, cols[col]); 
        }
    } else if(deviceOrientation == VERTICAL){ // orient the device vertically
        for (col = 0; col < myNumberOfDevices * 8; col++) {
            xcols[col] = 0; 
        }
        // little inefficient, can be enhanced, rotate the matrix !
        for (col = 0; col < myNumberOfDevices * 8; col++) {

            for(byte bits = 0; bits < 8; bits++) 
              xcols[col] |= ((cols[bits + 8*(col/8)] & (index << (col%8))) ?
                                     (B10000000 >> bits) : 0);  
            sendByte(col / 8, col % 8 + 1, xcols[col]);
        }
    }
}

[adityarao]

[witnessmenow]

@adityarao nice solution!

I'll update my PR with this solution as it uses less memory