Optimisations for beam.translate

ilent2 / ott

Optical Tweezers Toolbox (Version 1)

Other

51 stars 30 forks source link

Optimisations for beam.translate #30

Closed ilent2 closed 6 years ago

ilent2 commented 6 years ago

[x] Add A,B matrices as output to all translate functions
[x] Add option for output Nmax
[x] Modify translate_z to output non-square matrices
[x] Updates examples and maybe forcetorque to use optimisation

ilent2 commented 6 years ago

edbbac6 -- Add A,B matrices as output to all translate functions

ilent2 commented 6 years ago

6f6528d -- translate_z now runs much faster, also produces non-square matrices.

ilent2 commented 6 years ago

We could extend the scatter function to take a position and rotation argument. This would provide a clean way to hide this optimisation from the user.

ilent2 commented 6 years ago

d68cef5 -- done. Still need to optimise examples, they are running really slow.

an-kashchuk commented 4 years ago

The default run of beam_visualisation.m has an error. This command: slm_beam = beam.translateXyz([5; 0; 0]); doesn't copy the basis var to slm_beam and it contains 'regular' basis which creates an error in the following farfield calculation. This will fix it: slm_beam.basis = 'outgoing';

ilent2 commented 4 years ago

Good spot. Thanks, I've fixed it in commit 2cefab9396a8820313ced00f9650c1be392c127e