brainglobe / BrainGlobe

General information, resources and dicussions for the BrainGlobe project.
https://brainglobe.info/
9 stars 1 forks source link

[BUG] Circular dependency between `imio` and `brainglobe-utils` #64

Closed willGraham01 closed 4 months ago

willGraham01 commented 5 months ago

https://github.com/brainglobe/brainglobe-utils/pull/22 introduced a circular dependency between brainglobe-utils and imio: imio needs brainglobe-utils, whilst brainglobe-utils now needs imio due to the functionality that was refactored into it.

Needless to say, this is not ideal. On the surface everything "still works" since there are no circular imports, but naturally we probably want to fix this sooner rather than later.

Zulip discussion: https://brainglobe.zulipchat.com/#narrow/stream/408841-development/topic/.60imio.60.20and.20.60brainglobe-utils.60.20circular.20dependency/near/419020085

Task List (merge imio into brainglobe-utils)

willGraham01 commented 4 months ago

As agreed in the developer meeting on 08-02-2024, we will be merging imio into brainglobe-utils as a submodule.

Task list with updates has been added to the main issue comment for tracking.