OpenCOMPES / sed

Single Event Data Frame Processor: Backend to handle photoelectron resolved datastreams
https://opencompes.github.io/docs/sed/latest/
MIT License
3 stars 2 forks source link

Suggestion: Move imports to local scope #494

Open zain-sohail opened 3 months ago

zain-sohail commented 3 months ago

Due to how the current interface works, all possible dependencies are imported when importing SedProcessor from sed.

I propose we remove the linter for module level imports and import them in scope when necessary. This will reduce first time importing time from ~30 s to something more managable.

rettigl commented 1 month ago

For me, it takes typically 10s max to load. As anyways all calibrators are loaded with the processor, I don't see that we would gain much.