We have used high resolution transmission electron microscopy (HRTEM), aberration-corrected quantitative scanning transmission electron microscopy (Q-STEM), atom probe tomography (APT) and X-ray diffraction (XRD) to study the atomic structure of (0001) polar and (11-20) non-polar InGaN quantum wells (QWs). This paper provides an overview of the results. Polar (0001) InGaN in QWs is a random alloy, with In replacing Ga randomly. The InGaN QWs have atomic height interface steps, resulting in QW width fluctuations. The electrons are localised at the top QW interface by the built-in electric field and the well-width fluctuations, with a localisation energy of typically 20 meV. The holes are localised near the bottom QW interface, by indium fluctuations in the random alloy, with a localisation energy of typically 60 meV. On the other hand, the non-polar (11-20) InGaN QWs contain nanometre-scale indium-rich clusters which we suggest localise the carriers and produce longer wavelength (lower energy) emission than from random alloy non-polar InGaN QWs of the same average composition. The reason for the indium-rich clusters in non-polar (11-20) InGaN QWs is not yet clear, but may be connected to the lower QW growth temperature for the (11-20) InGaN QWs compared to the (0001) polar InGaN QWs.