The transistors are now connected backwards.
Yes, the 2N3904 or 2N4401 will do, it really isn't critical.
The voltage is too low for two LEDs connected in series, 0.5V isn't enough headroom and if you'd read my previous post, you would have realised that the batteries won't last very long.
I suggest you build the simple circuit, then go for the peak detector if you're not satisfied with the performance. No offence intended but you seem to be having difficulty with getting the simple circuit right which indicates that you need to gain more experience before moving on to something more complicated.
Yes, the 2N3904 or 2N4401 will do, it really isn't critical.
The voltage is too low for two LEDs connected in series, 0.5V isn't enough headroom and if you'd read my previous post, you would have realised that the batteries won't last very long.
I suggest you build the simple circuit, then go for the peak detector if you're not satisfied with the performance. No offence intended but you seem to be having difficulty with getting the simple circuit right which indicates that you need to gain more experience before moving on to something more complicated.