DiamondLightSource / httomo

High-throughput tomography pipeline
https://diamondlightsource.github.io/httomo/
Other
7 stars 4 forks source link

Docstring of `Section` incorrectly states methods in a section must be on the same platform #516

Closed yousefmoazzam closed 3 weeks ago

yousefmoazzam commented 3 weeks ago

As of #194 being fixed, a new section is not needed if only the "platform" that the method runs on changes (ie, going from a CPU method to a GPU method, or vice versa). The docstring of Section says otherwise and is misleading, so this should be updated: https://github.com/DiamondLightSource/httomo/blob/2e15a80e8e96753bc8a0d13285209cdbd9a56434/httomo/runner/section.py#L10-L12