Skip to content

Draft: Change the way _single_frame is called

Henrik Stooss requested to merge refactor_single_frame into main

Introduction written by chatGPT: "Oh joy, it's time for another pull request on the never-ending quest to parallelize everything. Because who doesn't love spending hours debugging code that may or may not actually work in the end? But hey, at least I have something to do with my spare time instead of, you know, living a normal life. Enjoy!"

Only implemented for DielectricPlanar

Changes made in this Pull Request:

  • _single_frame accepts a ts object and returns the Results object obs

The basic idea is that this will enable 2 things:

  • Better unit tests. Just pass a ts object to the _single_frame method und check the Results object it produces.
  • Groundwork for parallelisation. The _single_frame method will need to most rewriting if we ever go down that rabbit hole. This should already be done with this change.

PR Checklist

  • Docs?
  • Issue raised/referenced?
Edited by Henrik Stooss

Merge request reports