search

i4Ds / Karabo-Pipeline

The Karabo Pipeline can be used as Digital Twin for SKA
https://i4ds.github.io/Karabo-Pipeline/
MIT License
11 stars 4 forks source link

Rename `ObservationParallized` to `ObservationParallelized` or eliminate it #557

Open sfiruch opened 6 months ago

sfiruch commented 6 months ago

"Parallized" is not a word, we should rename the class.

Furthermore, do we even need it? I'd prefer if ObservationLong used parallelization automatically when reasonable. Science users shouldn't have to care how their workflows are executed on the underlying hardware. Every time we force people to think about execution instead of science, we fail.

sfiruch commented 6 months ago

Both points apply to InterferometerSimulation.__run_simulation_parallized_observation too.