I might have gone lower with the diode resistor for the opto, they are rated for up to 50ma, yours will be something less than 5ma taking in the voltage drop for the diode.
Max.
The 10K resistor (R2) might or might not be right, depending on the amount of light that passes from the LED to the transistor, and that depends on alignment and distance, neither of which have been mentioned. Then you have the issue of ambient light: if this rig must operate in daylight or incandescent illumination and there isn't any shielding of the receiver, you risk having the sensor say "I'm seeing light" all the time. If that's the situation, you'd most likely need to go to a modulated-light setup.