DiamondLightSource / DiamondLightSource/httomo

Remove unnecessary dependency of `mpi4py` in `standard_tomo.py`

Open
#350 0 comments 0 reactions 0 assignees View on GitHub
loader
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.