DiamondLightSource / DiamondLightSource/httomo
Remove unnecessary dependency of `mpi4py` in `standard_tomo.py`
- Dominant language
- Python
- Stars
- 10
- Forks
- 5
- Avg merge
- 5d 17h
- Merged PRs (30d)
- 1
Description
The MPI communicator was found to not be needed for parallel reads, hence there is no need for it to be there technically speaking. (Originally noted in https://github.com/DiamondLightSource/httomo/issues/208#issuecomment-1934087364)
Once #208 is done, this can be done.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing standard_tomo.py and the discussion in issue #208, which must be completed first. Confirm where the MPI communicator and mpi4py dependency are referenced for parallel reads; done means the unnecessary dependency is removed without affecting the intended reads.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100