The class excitondb.py is too complicated and convoluted, especially in the plotting part. I find all the plotting functions very difficult to follow.
I think it should be:
(i) rewritten for clarity
(ii) the plotting stuff separated in a different class
(iii) better commented what is happening inside the function and what is the logic
The class excitondb.py is too complicated and convoluted, especially in the plotting part. I find all the plotting functions very difficult to follow.
I think it should be:
(i) rewritten for clarity
(ii) the plotting stuff separated in a different class
(iii) better commented what is happening inside the function and what is the logic