As a precursor to asking people to review a project you owe it to them and to yourself to read and go through: https://easyeda.com/andyfierman/Welcome_to_EasyEDA-31e1288f882e49e582699b8eb7fe9b1f This will make the process of creating a design and putting it onto a PCB, easier and much more likely to be successful. :) * Two immediate observations. 1) Your project has two PCB designs in it. Which is the one you want to have reviewed? 2) First look at the schematic: 2.1) You have no supply decoupling. Anywhere! Search the Forum for `decoupling`. 2.2) You have no net labels so it is impossible to understand what connects where to any of the external circuitry. You know how it is all wired up but that's in your head not anyone elses. 2.3) No datasheets etc. for external bits and pieces so it is not possible to say if the external interfaces are fit for purpose.

Thank you! Both those PCB are the same, just moved them around, but will delete the "New PCB" Don't know what decoupling is, so I will follow your advise and search for this and implement this. No net labels, but do you need this on the schematic? I was hoping if somebody could see a "newbie" mistake I did. Forgot components, used the wrong type etc. The only thing that is external is the AC current sensor, I left a link for this, but this should be OK, I found this on google on how to connect it. But thank you for giving me these tips, I will read the "Welcome to EDA" you sent me. :)

`No net labels, but do you need this on the schematic?` Strongly advised. `I was hoping if somebody could see a "newbie" mistake I did. Forgot components, used the wrong type etc.` How can anyone give meaningful info about these aspects if all you have is a few components connected together and a PCB layout to match (BTW, well done: you got package assignments that pass into PCB OK) but no supporting documentation? Such as: What are the input signal and supply voltage tolerances? What's the spec of the solenoid? How do we know if a TIP120 is a suitable driver? What is the solenoid doing? Can it be a source of interference to the PCB? The outside world? Expected operating temperature range? What about this dust? Is the PCB to be protected against it? Are there any safety implications? To give you an idea of what you should have available when you ask anyone to review a design, here's an (incomplete) document about what needs to be supplied for a professional design review: What's the

Sorry: wrong paste buffer... https://docs.google.com/document/d/e/2PACX-1vRhMMBzCZ_orIFr56QyqYI8HcWsFA00evgh1cQ_069MJRU2YUJXGWzyX3mMVnxT-SNz1mydEicu6Bfh/pub

The datasheet for your choice of current monitor is rubbish. It says it is a **current** output device. This is supported by the fact that it shows no burden (load) resistance in the schematic diagram and has no spec on 'built in sampling resistance'. But then it says the output is 0 - 50m**V** for 0 - 100A. So that's a **voltage** output. Then it says 'turn ratio 100A:0.05A'. So that means for 100A in you get 50m**A** out. So that's a **current** output. If it's a current output then you need to limit the voltage swing at the outputs to prevent it blowing up whatever it's plugged into. Basically a current sensor like this behaves like a constant current source. A constant current source will drive a current into whatever load you connect across it so if you connect a 1mA source to a 1k resistor it will generate 1kV across it! That's why it has a bidirectional TVS in the schematic but the *clamp voltage is not specified*. So *you* need to provide that protection in your schematic. In practice the current sensor is more complex that my simple description because it is can only senss AC current and it behaves like a high pass filter. The lower the burden resistor, the lower the frequency AC it will respond to but correspondingly, the lower the voltage it can develop across the resistor (because the output still looks pretty much like a constant current source). And of course this spec does not state what the inductance of the sensor is so you do not know what the frequency response will be vs. burden resistor. I guess you have to assume it is designed for 50/60Hz... This is what a real current sensor (current transformer) datasheet should look like: http://www.farnell.com/datasheets/1790368.pdf?_ga=2.224850970.1856542880.1509439726-1003718477.1503994507&_gac=1.41027350.1508320034.EAIaIQobChMI8su--fH51gIVCjgbCh3KCwrxEAAYAiAAEgLbSfD_BwE or at least: http://www.farnell.com/datasheets/1870393.pdf?_ga=2.120960264.1856542880.1509439726-1003718477.1503994507&_gac=1.12265344.1508320034.EAIaIQobChMI8su--fH51gIVCjgbCh3KCwrxEAAYAiAAEgLbSfD_BwE

Oops. `...so if you connect a 1mA source to a 1k resistor it will generate 1kV across it!` Typo: `...so if you connect a 1A source to a 1k resistor it will generate 1kV across it!`