2014-07-22 18:08 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:
Hi,
It all looks good, but you just have a few fixes for the last few
commits with the broken equations in docstrings due to spacing
changes. You can find these with:
$ grep "\\\\$" lib/dispersion/*
$ grep "i = 2" lib/dispersion/*
This is done.
I have some more responses below:
- There are some empty lines straight after a 'for' loop in many places.
Done
There are a few more, just search for the text "for ei in
range(self.NE" in the target function.
These are gone.
I cant find any more left?
Searching for "for ei in "
- There are some FIXME comments in the lib.dispersion package.
They are not critical, and are in module which is not used yet.
Ok.
- In some places, comment lines require empty lines before them (in
target_functions.relax_disp and lib.dispersion).
Where? Can you give example?
You also fixed some of the empty lines before comments formatting, but
there are still some in target_function/relax_disp.py. For example
see "# Structure of r20a and r20b. The full and outer dimensions
structures.". There are a few in the __init__() method.
Now done.
- The newline spacing between target function methods needs some fixes.
Done?
It looks done :)
Cheers,
Edward