sebasj13 / topasgraphsim Goto Github PK

Show the calculated parameters in a table below the graph

Including:
*author
*version

Add option to change line style (solid, dashed, dotted, dashdot).

Self-explanatory

For single viewing only, when adding graphs to the view always normalize

import pymedphys

gamma = pymedphys.gamma(
(data[0][0],), data[0][1], (data[1][0],), data[1][1], 1, 1
)

valid_gamma = gamma[~np.isnan(gamma)]
pass_ratio = np.sum(valid_gamma <= 1) / len(valid_gamma)
print(pass_ratio)

Add "descriptors" + "Show parameter table" for measurements

Firstly, this is really nice! i have been vaguely contemplating if I should do something like this given the number of how 'how do I plot...' questions on the forums. Nice to see.
The following are some comments/questions/suggestions. Feel free to ignore any that aren't helpful!

comments

• I would move the install instructions closer to the top of the read me, because whenever I look at software on github the first thing I want to do is make sure I can install it before I start read the detailed instructions.

questions

• what scoring geometries does this handle? or will it handle anything? e.g. I think depending on the scoring component, the coordinates might be polar or cartesian (right?), will this handle both?
• Is it possible to run this 'headless' (no GUI?) and if not would it be difficult to add this? In many situations it would be nice to be able to script analysis...also for debugging this would be handy.

install experience

• In ubuntu, I had to do sudo apt install python3.9-dev before this package would install. It was some issue with a package called evdev
• In windows worked perfectly.

use experience

• in linux the default window is very large - I can make it larger, but I can't make it smaller. weird!
• I tried to load some .bin files from a TsBox scorer; in windows and linux I got the error "AttributeError: 'Simulation' object has no attribute 'direction'"
• In general it looks like there are a few if, elif statements where it would be a good idea to put a final else: raise NotImplementedError

[TODO] Use topas2numpy to allow for more diverse input files

Create a .profile with language preferences

ptw_import currently has an issue with .mcc files, that contain
INPLANE_AXIS=y CROSSPLANE_AXIS=x DEPTH_AXIS=z
instead of uppercase letters.

3ddose dataset shows y-axis label "mm" if not normalized. --> "Dose [Gy/history]"
Maybe add optionial or permanent label for relative dose distributions e.g. "Relative Dosis"