Since `core.uploader` is now only used by `pixl_export` should we just move it over from `core`? - Githubissues

UCLH-Foundry / PIXL

PIXL Image eXtraction Laboratory

Apache License 2.0

8 stars 0 forks source link

Since `core.uploader` is now only used by `pixl_export` should we just move it over from `core`? #384

Open jeremyestein opened 2 months ago

jeremyestein commented 2 months ago

Since core.uploader is now only used by pixl_export should we just move it over from core?

_Originally posted by @milanmlft in https://github.com/UCLH-Foundry/PIXL/pull/370#discussion_r1581279532_

See also the reply to that comment

milanmlft commented 2 months ago

Hmm upon closer inspection, core.uploader is also used in core.exports, which in turn is used in cli. So moving these bits over would require a bigger refactoring. Not sure if worth it at this point.