simpa icon indicating copy to clipboard operation
simpa copied to clipboard

T417 Adding possibility of wavelength-dependant laser energy

Open jeremieglt opened this issue 11 months ago • 0 comments

Please check the following before creating the pull request (PR):

  • [x] Did you run automatic tests?
  • [x] Did you run manual tests?
  • [x] Is the code provided in the PR still backwards compatible to previous SIMPA versions?

Code review The only modifications were made in the optical_adapter_base.py (adding possibility of selecting mutiple laser energies dependant on the wavelengths) and tags.py (modification of the description of the LASER_PULSE_ENERGY_IN_MILLIJOULE tag). There is also an addition of a test file (see next section).

Testing requirements The tests specific to this modification were written under : simpa/simpa_tests/automatic_tests/test_laser_energy.py. They ensure that all formats are possible for entering the laser energy.

Additional context Calibration study at iThera Medical with MSOT Acuity revealed that simulating with wavelength dependant laser energy could refine the simulation precision without any additional additional computing cost.

Issue fixed Fixes feature request #417

jeremieglt avatar Feb 04 '25 15:02 jeremieglt