atcollab / at

Accelerator Toolbox
Apache License 2.0
48 stars 31 forks source link

deprecates ncpu; use pool_size #631

Closed oscarxblanco closed 11 months ago

oscarxblanco commented 12 months ago

This PR deprecates the misleading parameter 'ncpu' used on frequency_maps.py, mentioned in https://github.com/atcollab/at/pull/608#discussion_r1271924788

Instead, an optional parameter pool_size is added with default value set to None. If ncpu is used, a warning is shown, ncpu is removed from the argumemts and its value is given to pool_size. pool_size is passed to patpass through kwargs.