Thanks for the quick reply, I did run the scanner and was getting the address 0x68, if I have just been using the 3.3V power output on the uno do I still need to use logic level shifters?
Yes, you really need to use logic level shifters, despite the many bad tutorials that suggest otherwise.
Better, use a 3.3V Arduino, then you don't need level shifters, or buy a more modern 9DOF sensor breakout that has the level shifters built in. The LSM9DS1 and similar work MUCH better than the discontinued MPU-9250.