Below are the comments I have recieved so far.
Professor's first review of Lisa Gerhardt's project,
"Manipulating Muon Events in AMANDA-II".
The main page of the project is overview of the project.
This lists the classes and gives descriptions of them.
It doesn't explain the algorithms and formulas used, unfortunately.
The rest of the web page layout is primitive and should be
improved. Besides the main page, there is just the program,
divided into many small files.
The program looked pretty good to me. I didn't like that it
was divided into so many files; it took lots of clicking to
look at it. No need to have only one class per file.
main.cc is horrible to read. No significant indenting, no blank lines
for spacing; it looks like a jumble.
In general, the program can be formatted much better. Some lines
are huge in Probability.cc, for example.
Although there is an example main program to illustrate the library,
but there is no detailed explanation of it.
Overall, the project is OK, but it could use some improvement,
particular in the Web page layout and user-friendliness.
My response:
Formulas have been added in the section describing the probability class.
Unfortunately most of these formulas are the result of personal
communications with other people in the AMANDA collaboration, so
there are no references to give.
The multiple files have been joined into two files called Classes.cc and
Classes.h. Now there are just 6 things to download.
Main.cc has been altered to make it a little more readable. Returns have
been added between each example and the comments have been changed so
hopefully now they will stand out a little more. Also the long lines that
extended off the screen have been fixed. A short section describing the
main.cc has been added to the web page.