Arduino2Max Problems

Hi, I'm kind of a noob with Arduino.

I'm trying to create an Interface for Max/MSP where I use sensors to read data in and then use PWM outs to feed data out and back into the inputs.

I'm testing it by simply using one input, mapping it to an oscillator and at the same time dimming an LED. This isn't working though.
If anyone can help I'd be eternally grateful.

Cheers, Jan

Arduino:

 /*  
 *  Arduino2Max
 *  Send pin values from Arduino to MAX/MSP
 *  
 *  Arduino2Max.pde
 *  ------------  
 *  This version: .5, November 29, 2010
 *  ------------
 *  Copyleft: use as you like
 *  by Daniel Jolliffe
 *  Based on a sketch and patch by Thomas Ouellet Fredericks  tof.danslchamp.org
 *  
 */
 
//=========================Declare Global Variables====================================
int x = 0;                              // a place to hold pin values
int serialvalue;    //a place to hold the serial.read values
const int ledPin = 9;
//==========================setup()==========================================
void setup()
{
  Serial.begin(9600);               // 115200 is the default Arduino Bluetooth speed
  digitalWrite(13,HIGH);              ///startup blink
  delay(600);
  digitalWrite(13,LOW);
  pinMode(12,OUTPUT);                //Declare all of the pins that you'll use as OUTPUTs here
  pinMode(13,OUTPUT);
  pinMode(9,OUTPUT);
}
//===========================loop()============================================
void loop()

{ 
  byte brightness;

if (Serial.available() > 0){         // Check serial buffer for characters
    serialvalue = Serial.read();    

   
    if (serialvalue == 'r') {       // If an 'r' is received then read the pins (r = ascii code 114)
    
    
      for (int pin= 0; pin<=5; pin++){      // Read and send analog pins 0-5
          x = map(analogRead(pin), 1023, 0, 0, 1023);
         
          sendValue (x);
          }



      for (int pin= 2; pin<=11; pin++){     // Read and send digital pins 2-11.  Modify this so you're
          x = digitalRead(pin);             // only reading the pins that aren't used below for digital out 
          sendValue (x);
          }

        Serial.println();               // Send a carriage returnt to mark end of pin data. 
        delay (5);                      // add a delay to prevent crashing/overloading of the serial port
  
  }//end of if (serialvalue == 'r')

      /*------------------------Here we receive the numbers sent from the Max digitalOuts-------------------  
      The digital writes and reads here are meant only as examples.   You can replace what is in these
      and expand this whole section to deal with more incoming numbers.  You can use any number from
      0-255 except 114, since that is the ASCII code for 'r' and is used above
      Be sure any pin you use here is declared in setup() as an OUTPUT and that you're not reading from
      it just above this section.*/
 
        if(serialvalue == 0){
          digitalWrite(12, LOW);
        }
        else if(serialvalue == 1){
          digitalWrite(13, HIGH);
        }
        else if(serialvalue == 2){
          digitalWrite(12, LOW);
        }
        else if(serialvalue == 3){
          digitalWrite(13, HIGH);
        }
        
         if (Serial.available()) {
    // read the most recent byte (which will be from 0 to 255):
    brightness = Serial.read();
    // set the brightness of the LED:
    analogWrite(ledPin, brightness);
  }
       //--------------------------End of receiving numbers---------------------------------  

 }//end of if (Serial.available() > 0
}

void sendValue (int x){              // function to send the pin value followed by a "space". 
 Serial.print(x);
 Serial.write(byte(32)); 
}

This is what I've done in Max so far just trying to get it to work:

----------begin_max5_patcher----------
6538.3oc6cs9aiiij+yc+WAWueX2EvcNQp2y8o416ENfF6fsm6vcX5EMTrUb
zzxRFRxclLC16u8kjEksrsDUIaIEEGOORhjskI+UEKVrd9au+cytO8WBymQ9
NxOQd269s2+t2Iuk3FuSc86lsN3WVDGjKeayR1t99vrYygWZSPwhGiRV8krv
EEvig4ytyXNw1xS9KCwOo92YP9apOSzR4CJ89e9C19kOnGRSJRBVGJeo+mvr
kAIA6+Rx3uRQX1WBSBtOV9dLTuFe3DkDGVHGbz82LcaQ4cYU9Jxi9U4GmJFV
vsg2YwyaBgw+rnjhYyIytOHY0LxeS7l96u+8heLGIDkGGsDCDY5KgHF.QVFU
vnxwIy1Y2.89UKRiSyfGgwc9V9Fr409G0h0tF8DdRaB3NOzJI7I9vqbrUD9K
E.HFFywKhAgpAHsDLVyIVNRfzgUBm0N+cqkW66yhBhmU2r2r1YuYcbSrl4lj
rQy2+6yCjVjtdcnfu7HTZQbzhuRdLLKjTjRVDmly+iGCI4ghoEYSZVgN3ywU
tF01G.NGc3mmQGwu54dLZ.+FLPIcSXRWvDk3KSSLXBcpgIqCyyCVEVClvYMZ
ekjhUvT9SpWCyZVGm0LsxQvtR57V4zDfHXCHA5PDKZEFA3hFQDqWQHRSKa9y
GrrYUHGdHwQ4EjzGpt1IWClYSgEOFtvpFWsKdrmZKdJRWsJFwpDKSfavt7m0
N8bG58aEJpzuqIxhRzueAsB40zQ6RBuWQKIZTGDIWe.w2wvPCtXBPgsmTTgm
kVtd+dgqm0QbQoT6PHwfKrHuHP+lpVNUW2P0JWvexso5lfjvX75yCSRyFNyi
OS6jHZURJ+IKP1p2dUVvRJ7sAO9p+X2WS8Hf3yxJO1fsgumg07Z+qpOp0oKC
qN.3puBRtXtFmEH19AGMjKbL8pbvwFWE4zNOxPbrQ1.drwvecYvh+eM.jiAn
blg4c17ESxcerZX2GVGjwZziRLCdR2Lv1EnpLm8ZY1DIl9hr+QNe8G+KnO2E
IiDX0tvClgjdx.Muca.SL6HnXfFT7Go8T4ngs1sRcwiFVWCngoNdCKe7ng80
.ZvztAgMdzv4Z.MzY6KJnLNNzv8Z.MzoBNk5fGM7dEgFGn0ziB33.q.64ZX6
4Mu1+pp5bZfLGP2biSNM2QFb1yg+Otyq8un0izVVuLpog159kbYYQ4KBTe22
4OwnSpE5CIcx9Fc5hoSkxdFR5jyM5zkSmTJUMjzI2azoKmNoNXzPRm7tQmtX
5T4Q1FR5j+aE5TSlZ8uFFrjC6RGY98YK2Fkj9GxIAIAwoqHahRz4UFpIXQZe
vklL6RqTVaHYvvDRFiq8Wa7HAB+QEk7PpNSO6A1Zx.BJEcmKneO.8iQKWFlL
ZGTXKmsYa63fsKDrImd.o3f6CiOwtyUfmKT2iSD4XXwku3KiWEoH.SeF04D6
Y2FalY2.2R+f.9BobRVDtVEDTy9Wh2FVjlV73G9g+0u+Ce54jEh26bwO1+Re
LcY359991pe6n9s6YwGzpOLUp+3.RAXk+r1EDFSVWX1jbx17DEyRJBvhseup
lL.s8jyOTMHGbCI.1Q3SR+Ulw2pHLiyZ0NS.kp9ksV4hl8ho3MztRUNHCyTy
d0zm+lihC+VXVdTZUQouaVvlMUt86p7QDX1OCRYblu6VbzIqxffeqrvuEU94
MfaxoDuu7E2s4ojGAzsgZB9uvSDUdkHkP+l3zEeMbY0m+LQXGEkrIKLmylFT
T9EU9xKCeHXabwWp2uDG95ODrHrwObsjCgO+hVllHFDG7IE2t7q6mpI.FTui
jfM07g4qsJ1leePl.3TBAXkuHWfV7guzNZEmoNHIZcPQXQDLXYF6dnQqgnNn
5WDHi4w7EYowwG7nfW4a07JK4jyEgOEsr3QP6qJnE+sGsojDLaGFsLZUXdwg
2qHXU9g24fHjsJeV0UqGbecqZOVClGCi2T8kpYspuKv74SqD0EUVp1jiyzuh
sQ0YZV7cyqd0pHy6JEzoVh0+vXNDXRxHFsEvjRcMqXnXGqZQyCOuhP6DG9Gz
wV7WldFbMUNPOklb9ZWo.zFo.FMSAFGHNCODaBZV.bsCC.a9FkEuxYcZbmcv
2GVVt0efmSNZ80Gu5FRNGq1Ysi1PpxXVbHXTs6cz87YTOFM1qr0g3biJc0jh
WG8wqSArFTBqIEwNPYriUHCBxNYPBY4R2ExPUmf0qSFB8xvnaVq5mgTGMM5o
gSWsVzWqUc1ZQusV0cqI82zqCmd83zqKmV84ZRmt50qqdc6NlU9XoQm75GIU
BV2U86nwCfI2jzxYeF.U8bnsoWmVIF5kIqDqAGeVhHUeocRlOT57khKxwIZX
wjgGUXZQECsnxwXldoluH.2IaycbDaGxW0ujjyWdVDtl3h.iMg3rj5Cmjf41
HLeFa9oSSs12Drqr0zZd3MHwtM02lNzPGLqSJ0q1XDngV2ngckFZigFBB6nP
f7NvzP6azvtRCsPPCoPxdTZG0gkF5biF1UZnIFZnkIren6HPCcuQC6JMjggF
pzoANO3.SC8tQC6JMjhfFpTowZLTK0+FIrqjPCzGeixFCsRoF2ngsQC2lrIX
wWI4G9unIjd9UianghNRGT5n2kRGmq4+mBz3L9PNrZUsnc5JTjGFXxJ6BIqL
sjUVePVGEx2gQpxozuOFjDrJLurDKHPHxxfhf6H+aAKdTRXga9G2tQjq0bxQ
jnflDkSVDjG9mHb4zwAKBEQCHQJoNBp2IKyR27gkoOkPDgFlnJNHt6lfL9ng
H4MH+QYjOI9.6ey7Wm.QKZ3RhA4CD2+zcGO3iiRBWjtErClEB0yfBMCS4ZP6
ckqGsbPmgsiNMtVwy7dAlSsEtuGxVcOlAjFtwCM5u.+awt0Rpf38omUMOca1
hxEEJMiHmhMKCyKhR14tjeZmEoq48tOvGM5IwJKixEtKX4QdHpCKYOWzv6FZ
TYF5dCMpLCctgFUlg12PiJyPqanQkYn4MznxLj0Ezfc0CGzNgGW+rGxoHEId
3+1.NXHgCu2FvgIR3v8sAbXgDNbda.G1HgC62FvgCR3v5sAb3hDNLea.GnU7
v35GOLdEql9g1Gaoz0UgIKd9KKBV7nZVVw.aUCt3Y4AeKb4W3ypvEEeInnHK
59sEf80pK5naJDSQFwus7x0Gnp3BIYMwE6p3z6ChUw88tu9SFX0FJycNNne+
oDoANh+U9p7Hufzb1oXBkvVHkp56f92YBEz+8H9yWTTvYsZCdgn1ERyVkGf0
j9ItWHZTe81uWm2m5.m8LdeJr.ps7Aq2DGRxBJB+Nx8gEOEFlPD4PwyDp8Zg
qYVVdogw576ZCE8AGoq5nAdslEOduVyhmpkkDXKjyszjzBGI0EJZ010BkmWI
JQWTkzBUPSB02EpD67DLTWwKAXrOs.lzST5CZJJsQqTAPhCTuaOJ4d10bT7b
NXV1Trgb9XcZVT0jDBmDI9yRdK6IRlCFmFrDZ9H5yFNW6JBtgl2i1rLlNU1k
q+k3qAMWFFG77AgFb8kBNnRfCt8VEeo5.yyMigYuB.ygWDuEjlq1drAWDOy9
lL9WzcygXRaTH0N2H0unjZ13sp18Fo9kjTyrGuU0d2H0ujjZp+3sp914wdYI
0rQaUsowMR8YZ.KYTG+ChnL9S7AJmBl2pwojTRndRxLa0DelzQy3ThxN6HYw
4E7u1rzXb0tKlpdC511AAMYWm1N9vNN2gPortkfCFgVqETKWOtTnb.LZNUNO
8nYXhcEARF+S7E7kUtxJIfgQq7liZoh5JoZEY4YVIwTcgNemsH0atUuhtUuh
dwqWQskbRUKmt5O.mxgU1UrdrWGqmQ8Zd+oMEcLtyXBj4W4mpuUiV3DJRcJG
BZ6fAeYW636INjtQMVU8yZev2zlk+raEtnpGTgppY49FTeGYILW8WWXhk0ZE
o5H6y+Bx8djJFMi8vFANlUNDn2KYAG5UfjAUGdf8wfegrMGRsS08H2mxUzir
IKcE+XrqCWRdJp3wp8EBxhzkg7aDTPdRp4T9QcMBt5DaKjMMBYbIrN3qp2xF
QhfFTDPB9VPTrXGO9syR2t5QBmfFF8sPBDkThO3yOI5T37Owyoayf7IUedhx
vvrH0RBpKmP8Ii4+RVXidEvqv0fHpPza122DPzAw1VplHKzE.pM.nzEPSiKF
6OIf3SayJ5PXOiJ.rKJ.18JGfQuSsEzbZ5zN0du02o9fV9gdszUU5J199TNq
qUJmVLeZuii6aQHCLL1V8voz.KP+0.ObWd1HOSLhBt3xZi4kHKntI0gVd4TT
TqEXzYElZdTMYMFMVjQmUYNwxLGYclO.UQZUUKBrqsimyoTnlMPCRizf0PMn
LVSGLXSKFsAugaPX7FTFvAgQbPYHGcFyocC5ztQcZ2vNsZbGcF3oYi7zrgdp
a8RcB4p88gnfL2jbLGn62.AylF1iVruyohpLp+Mz7tCZ2osl2Z0p0r7qUT.c
Drtx9tBbnkYG+wNX6kS2h4khJPolU6nFnICroHY3gT0QFCyHkdR6UBc.5eLJ
KJglJXNVTAPcoVHBTBG+MDKFjF38eJuHcSYAnRndSNFhwoaOTorDc5K1ZNYU
2l0XxMq5zHQmT2NrwIh2RyxvwuaeKagzdtaoM+sZKGtPjGWmxA7RYXLxCgOQ
RR4r.eG4Cj+6js4gKACZkSdJJNl7P71EEa4pMrKMb9VP71v76H+nnjow+OtR
jqChmSBxIaS3Be3bV7mgzbXKSgBqVprBpEkQReJ4NLF4Db6gx6ddnLxI8Mu+
O9KIKBEVOj7HeQtz.jA2m9sPXig4xWYQPBYEe+hSZ4sBCWVZ4Kf.ehIIEEyt
x8Xt+YgYNEMP2.xmmUZLyfe+mmIjDt+NKk2QJsYN4oGEOteufmQL31Y1T4H6
ofDo1DBwk5s6oIlxHNXTNHSALo0l2KZSZfqQqF0VgujrDS481AhrASyclSFw
gvuz1VgwYWaY8ekTZY4nOCk4ls5N5acC8aE8CvTRsory.8sug9si9XpF1vAl
6F36bC7EU6A7BV12fIa.cOJ.iMsY9Vyq4OZqdF69lwP33H.k6qNZD.udm.jl
brihTdKxnQ2Coif8vCm52ImcQwdCOyWP+cbbF.b4YAPKRBAVEnqqW6wituFO
24ZIp8FUKBGswnbFM2gddsLqkCeggGXdCVTqoDE35fSoTiktwpzo7F4DCjZb
ieYbO3+GHUCvorsI4jGhCdJNLOO9YgwWB3zLOSx+w+4uR9Xvh6SS+J4GxR+m
+bl7f8qihCCVExOQ+y7y9m87kG1Qf4affqzw1tLICzx07l27Mef7CxlHPvxf
MBKn8PV5ZxO9X55fbxeYaXrvN++6YgKCyhV707+.4yy9TjrJ47QHYPHe54hv
0edFI7WjUOm6HpjQ5ornBg8bxDs0hboKBmSteagvteD4iPLt3eiB6y7cj7Pv
3Qe5tOd2mtSF8aBi8Ej70bgAZ9TX3OyYT9u3TFgg8f26G+9+WRbTN3HBYnus
MNtRyRP7P9cWLakCDGaxMxbLQENaL1M1p+uzsx01fYASIK3zxUgfQ8TNoP1U
LDtuXaDT.lj2HJQDsj2Q9ywbNNkQbIOy4DSeh.gfj7yn3WVIrYrpqXTw2GRS
KxeahvhT8UK4idj+PfFngfouHHFXup7oEig9RVDDIjNfAAakow7Me.yVv2UH
IZ810fciSB4hGDjlOO6fn1gKsozjvA.iSVzpGk15MhSUWlphD1mRy9pf2P7o
DVMVEcrRuHH3IEAbEoXaVR9AtMCjAVMVbEalw4m3mMPOmga6bFp9SnqpcgxJ
CUVsbFVuwCcyerjD+gGEzgCEFHhDf7JKpySgvltzyCAK+4s78HJySi47sL3x
DDBBRenHLoz7+mR0EeU.GUofJgboKkAvxWkB0fojfVdXqb.1uw4.pnp4LrQf
N7SUNKvZwlcLmdnOEY3wLsMEmZgQ8cLjGcxPigQ5Kw2dZaVQsNpl.z2+pXEn
XwmSUeCRY6bOXkzcPsae4xT9wlREpBhwEuLIToh6CSSKTYfk6099xaBRBiQ0
mj2aCbGUSK2fhzBP9NdNL3D83r.D6DSElwEytr1f2TOfGsRn4vBg9jHnGi.f
WWjJUq4wgxrBDi2MGlRMVtTtTOKruRPSQ3UgSJVz6CUoYSNtXUtX5nRsCp0P
wFaZb0xFiEwUADgIj8qLI9aivIC1F9dFRmK3Ya4avj3u5dsg5z27nNyRIv.j
gvTMFc+gE2Yu4wcaPPsoxWwftFCM6t4jC1ml85xt1.HVGsbSJWCS0Xm4vtyV
rXBVSU4pWqs6RiZaLSnADKtNzRH.rRJ+WkW8Zsy0YTai6pY.4ErUezodWYWa
xXufyqtzEJYFudlWcpeR595Yd0o0WdCz75Vui4sbuiIijlj9vCsTszTEJAnZ
GZQasVoYel0JMiocAn6XyOTehpAoHE0xoYiOb.X4bgvQqIAUGy.sAtlaJy8P
gQJO56mjWr8P+DUuaf.aoAoL.0o0N0fo6qzlCCJlM0YkXFH407FUdM7IZ2.y
y8Ch7jlD.A8P5CUcY8N2XyeXjmhRVl9TqU.VnnuZCUPh16uEl9uRYAQ0yqJ6
1G9933AsLl9M8JsUZTt7puElgboICpkK1sxjXQmP091qjxOpZWBHV48L8f.b
y+V0G8V0G8cu7UeTXgGBCiBRVMoN38lEsGcr5X6OpVR+n7MgwnbWEzjKYTHQ
MPUIxF1vO7By+nQK0HNsbkeHEfhIwSU0vL3vhLCLLsWZb7wt5S8tBx8jXzfO
ceCHscV+gMT4XWF3CcvuwhHfMJC.T10.uXY6dLFC5wHIXzb4TWLPs0qF631E
ySadY9QiqJfvOQPXR6CG7xzdR4zHqZ8CTCNg3UCM1b.6N70SigNLCUc5Zuok
mAYiGbPoN8MdbykGukc4QNInsNXqpOa5nrzWsEufYmFialNFtdx.QwQE5vZZ
ZW1VuV6Z4ZgV6Yn5VsCKzZeUBslyP0xPGVn04pDZYyP0hNGVn08pDZoyP0RL
GVn06pDZMv4ChgEZ8u9f1c9349fEeEGFqBKd2V8yiiwM+7z294Q0iUoJ9bOJ
DSk2Zyb2bzyTvQOsXTWQ9yl+756SQ4UCUGmkUsZX69R1p4tFrqdTQZ.dvmBh
arLFCWJYdU3RoVv+eMlrJKc6FhqWGHCPiVDWSbZX8szE5diwy0Fs4a0vXB0f
fxAeJpfmUyw2Re6iI8qEL6GeLMt9ZBcP.TJ3WYzemxed9ta57iB.9aa1DxOb
JnwzEui3bFRGwUWSH3nXvcR3MNmqVuwQGGuwIJQFhDYFr6lHhuKuZh4ON1Hg
GdFSe7nKdx60i+IGRGx8BlgTcXVYeK+nt4rv9O+n36tGJya.j1ADBGHaVq1A
7bi2aVi1Ajcl1AbdemmTZvSU0Lz.oWBofo7FDKW61+Nf077o.phX6gmHeRPK
JcF9fRKruQKPQKXiv5BmazBLzhReBOnzB2azBLzhRmHOnzBuazBTzB1Hrtv+
MNsXaxlfEekDs+ewQUrgpHlokUaJn50+Np147f3Jvqte0qP+o4.SkyDfL0OU
NHxosLh0q+OJvjnbI.EJZjovtsKtzG16nP0RamnnWRu3wbYMetjkRnFHkwxf
pfrcqUjCOyoBK1nlr1p77Of36XXP7HTradY5HYDU8xTc.a+eHR1EIkb7DCBg
3QKmWwFx9RCJTZ+Zc0s80onviqAh0e1NoHPWGnRKC94y9Xv5nZenytJSsmqk
MUV5C8EZRoU8IuCNhW8k7vlYWatTG1TYNrBBtaHcpms1Cp0Yu3JPa6kxvFrQ
rPL3gNGpdCDeriMLsk4jCUUiKbM1c0geo0YRYsbOcch5YicldhC65zL0TUjm
L81c0XOSsPNScLPNSGowLEwX1d5LjMwxPQe8MloryaQfhsmA6Ww36+Wd0XuH
f0EAaSDpCE4X1ZBwQYfbLaZLsFyXD1v7mViYFlwr2zZLahYL6NsFyVXFyNSq
wrMlw7zY2SWrJdZ6eQ6D4.U8vpWMx6D45gcl5cdyTlgLVpb7f5FSkqF6YpK1
Yp6YNSc7pNSqb0XOScvNScNyCSXVklV8pwdlh8XS1m6wl7qRSqd0XOSwdrIa
6yalZ4TklV8pwdlhU03oy1ENXUM1a5nloCVUiclNpy6z2FIY3GxXEPMcFwVz
Wenrkw.a9LgsBTa0Uw7YllitzQSr5q4wlLTGSrqamPCY69FlOhgRYpYnqrSg
RXK8DmBLBSTr1byT2DkNtiYrVOyzZ5LlwJgxc5XUGrrFSGaMvvdtc2oicnXX
kn6NcrCECqR5tSHdCr6B4NcNXAC61PtSGcwnXkNaMcNLDk12G57X8G8f1BGT
BSTNiUd0HueOE69P9WjeloNtUlnxqF6IJVcD5KOrsO.HzlKYk4QVa4P1u89F
ycLD4MllPr4z7EqsbEqw7DqkbDq17CaVyeIMkWX.HK+oBlOLFSjP0ILH+16a
veDGreaSqyOf8+TNhSrdFnOuHJbkwUg8tq1+EcL+iZBMu6y.WDyfC0V.wL.B
fXKH4GrfIhU0lpQeNCbvLCLZYFziiGqgfmvF3IrcpxSHtZHPT1PLC.t.aW+p
y.wUCvLPxvRaaFXMZ7DtXVkcDrOniGCDiGa+Qa7fYIr8Awk2fNbNZl2OK.Dw
2pjk25fE.VCxB.aCrHZGlALa4R3xVa1tKFhwOhguSCidZuOZrPIR2YzVtfZ7
PGsgCEiVPVimzVJFosV1i23Ai3Mqwi8ghY0kk63MdvvNaMd6FcXOWtowi23M
dPLbNWcG3mj7u+9+AEXDFkA
-----------end_max5_patcher-----------
          x = map(analogRead(pin), 1023, 0, 0, 1023);

This is an expensive way to accomplish:

 x = 1023 - analogRead(pin);
        if(serialvalue == 0){
          digitalWrite(12, LOW);
        }
        else if(serialvalue == 1){
          digitalWrite(13, HIGH);
        }
        else if(serialvalue == 2){
          digitalWrite(12, LOW);
        }
        else if(serialvalue == 3){
          digitalWrite(13, HIGH);
        }

So, you are sending 0, 1, 2, 3, or 'r'. Really?

Or, are you sending '0', '1', '2', '3', or 'r'?

void sendValue (int x){              // function to send the pin value followed by a "space". 
 Serial.print(x);
 Serial.write(byte(32)); 
}

Serial.print(" "); would be a whole lot clearer.

You know that this is converting the integer to a string, right? Are you expecting a string in Max?

I don't really know, I just ripped that code out of the example that comes with Max2Arduino. I've barely used Arduino before. I have my sensor built and it is being read into Max just fine (cheers for the tip about inverting values though). I want to use Analog Write instead of the Digital Out stuff, to feed values back from the Max Patch back into the sensors.

Thanks, Jan

I believe it's an integer I'm after since I'm reading the value in an integer box in Max.

I'm still stuck getting Analog Write to work. If I use the 'serial out' part of the patch in isolation it works with Arduino and I can use a Max slider to fade an LED. I believe the example tutorial is called Dimmer. I am essentially trying to integrate this into my project.

I have a sensor that feeds data in to Max from the Arduino this data then needs to be transmitted back to the Arduino in a feedback loop so I can modulate the voltage coming from the sensor (I will run PWM through a LPF to convert to analog voltage once I have it working properly). So I need to use Analog Write on PWM outs. But this isn't working. I have an LED attached to pin 9 then to ground and it is not dimming.

Can someone please help me with the code, I have no idea what I'm doing! :~

:grin:

It's ok, I've got it working now. Was simply a matter of me not understanding how serial communication works in Arduino. Hooray for books!

I wrote that Arduino2Max code package about five years ago.. It's amazing that people are still using it!

If you're reading this, and know how to code, fix it up and I'll release it as a new version. Otherwise, I'll just enjoy watching.

Daniel