Electronic – How to do indirect addressing in PIC ASM

You cannot use sprintf() to format a float into a string on the Arduino.

Not only is the format of the command wrong, but sprintf on the Arduino does not support floats.

There is a function dtostrf() which converts a float into a character array. You can use it like:

float temperature = 85.4;
char temp[10];
String tempAsString;
String jsonData;

dtostrf(temperature,1,2,temp);
tempAsString = String(temp);

jsonData = "{'version':'1.0.0', 'datastreams':[{'id':'" +
    datastream1 + "', 'current_value':'" + tempAsString + "'}]}\n";

The format of the dtostrf function is:

dtostrf(value, width, precision, output);

where value is the float value you wish to convert, width is the minimum number of characters to display before the decimal point (padding with spaces as needed), precision is the number of decimal places, and output is the character array to put the results in.

1. Firstly, you always need series resistors with the LEDs when driven from a voltage source (e.g. battery, DC supply, etc)
   This is because LEDs have a non-linear I-V curve, which looks like a high impedance up to the threshold voltage of the LED then rises very sharply, so it means with a very slight change of voltage the current changes a lot, making it almost impossible to set the current to a stable value in this manner.
   By using the correct value series resistors, you ensure the current cannot rise enough to damage the LED.
   To calculate the resistor value you need to know the LED forward voltage (Vf) then subtract Vf from supply voltage and divide by desired current, e.g for a 5V supply, a 2V Vf and 15mA:

   (5V - 2V) / 0.020A = 200Ω (The standard value of 220Ω will do - if you don't have this then aim for anything between 150Ω and 600Ω for a 20mA to 5mA range)
   This is assuming a typical 5mm or 3mm LED of 20mA max operating current.

2. Although it says "3 x 1.5V Battery in Series" in your schematic, the batteries look like they may be actually connected in parallel. T confirm, the batteries need to be connected end to end like the bottom diagram in this image:

   

3. You need decoupling capacitors present between the microcontroller Vdd and ground. I won't go into detail (search on here, there are many good answers on this subject) but they are basically to provide the microcontroller with a low impedance local energy reserve for high frequency current demand, which the power supply cannot respond to quickly enough.
   Ideally you should place one (100nF or 1uF ceramic is pretty standard) across the power and ground pins, as near to the pins as possible.

4. Make sure you are using the internal oscillator if you have no crystal present. Your code does not show the config bit settings, if you have left them out you need to add them to ensure the microcontroller is setup correctly. The CCS manual should tell you how to do this. Also in the config bits, make sure the Watchdog Timer is set to off, otherwise your micro will continually reset (unless you call the WDT clear command regularly)

5. Make sure that you have the LEDs the right way round.

6. Make sure you have the MCLR pin tied high, otherwise your PIC will be held in reset (unless you turn MCLR off in the config bits) This is usually done with a resistor to Vdd, value around 10kΩ. The datasheet will have an example of this in the reset section. (thanks to ajs410 for mentioning this)