Greetings!
I've lost loads of sleep researching quite a bit of odds and ends of electronic components, getting the courage up to post a schematic for criticism. I think I'm close to having an acceptable design, but having zero experience with op amps and MOSFETs really puts a damper on confidence. So in an effort to curb rising "magic smoke" levels in the atmosphere, I thought I'd ask for some criticism on a circuit.
My schematic so far:

(Click to enlarge)
Notes:
AWSP40-24 is the 24VDC 40W PSU being used (Forces a big RC time constant...but I can live with it. Might end up using only lower voltages most of the time for battery tabbing)
IC1 - OKI-78SR-5/1.5-W36-C is a DC/DC buck converter outputting 5.0V
IC2A - LM393N is a comparator with an open collector
T1 - 50RIA20 is a thyristor triggered by a footswitch
R2 is overkill in case the short formed by the electrodes during welding is not removed for an extended period of time (but this edge case will generally be avoided)
My biggest concern is whether or not charging of the capacitor will work properly. IC2A (LM393N, comparator) is supposed to compare the voltage set by the voltage divider with that of the capacitor. If it is lower, then it should output 5V to Q1 (IRFP2907ZPBF, N-Channel MOSFET) to charge the capacitor. In the other case, IC2A is supposed to output 0V to Q1 to stop charging.
The idea is that I can charge the capacitor anywhere from 0V to 20V, but using 24V to decrease the RC time constant.
So, does this actually do what I think it does?
Also, please don't be afraid to point out "obvious" things to me - any appearance of experience on my part is purely incidental. For example: All bypass capacitors I believe I need to use (all one of them
) are on the schematic. No fuses/protective diodes are planned to be added (yet?).
I've lost loads of sleep researching quite a bit of odds and ends of electronic components, getting the courage up to post a schematic for criticism. I think I'm close to having an acceptable design, but having zero experience with op amps and MOSFETs really puts a damper on confidence. So in an effort to curb rising "magic smoke" levels in the atmosphere, I thought I'd ask for some criticism on a circuit.
My schematic so far:

(Click to enlarge)
Notes:
AWSP40-24 is the 24VDC 40W PSU being used (Forces a big RC time constant...but I can live with it. Might end up using only lower voltages most of the time for battery tabbing)
IC1 - OKI-78SR-5/1.5-W36-C is a DC/DC buck converter outputting 5.0V
IC2A - LM393N is a comparator with an open collector
T1 - 50RIA20 is a thyristor triggered by a footswitch
R2 is overkill in case the short formed by the electrodes during welding is not removed for an extended period of time (but this edge case will generally be avoided)
My biggest concern is whether or not charging of the capacitor will work properly. IC2A (LM393N, comparator) is supposed to compare the voltage set by the voltage divider with that of the capacitor. If it is lower, then it should output 5V to Q1 (IRFP2907ZPBF, N-Channel MOSFET) to charge the capacitor. In the other case, IC2A is supposed to output 0V to Q1 to stop charging.
The idea is that I can charge the capacitor anywhere from 0V to 20V, but using 24V to decrease the RC time constant.
So, does this actually do what I think it does?
Also, please don't be afraid to point out "obvious" things to me - any appearance of experience on my part is purely incidental. For example: All bypass capacitors I believe I need to use (all one of them