Function call needed instead of reference.

(and fix viewport scaling for non square viewers)

This also factorize the loading codepaths with and without slicing

Just checking boolean value would lead to a len() call, which might fail
as line_idxs might not have been set yet

self.layers dictionnary could change from another thread, thus changing
size during repaint iteration. Using a fixed range with the current
number of layers avoids iterating directly on the dictionnary.

Now all we need is to find how to handle the numpy.fromiter calls in the
3D viewer to have the view actually show something before the end of the
loading.