út 12. 12. 2023 v 22:46 odesílatel John Kacur <jkacur@xxxxxxxxxx> napsal: > > If you're removing the CpuList class then CpuList, or Cpulist is available > instead of cpulist_utils > cpulist_utils is a module, unlike CpuList which is a class; these have different spelling conventions for case, see https://peps.python.org/pep-0008/#package-and-module-names. > > Instead of removing these, you could change them to > > compress_cpulist = cpulist_utils.compress_cpulist > expand_cpulist = cpulist_utils.expand_cpulist > > and then you don't need to make all of the rest of > the related changes in the file > Or I could directly import them like this: from rteval.cpulist_utils import compress_cpulist, expand_cpulist The reason why I did the change like this was that I wanted the usage of cpulist_utils functions to be consistent throughout the entire rteval source code. In systopology.py and other files, CpuList.* is used directly (which became cpulist_utils.* in this patch). Tomas
