rename cpus_by_node to cpu_ids_by_node

Author: tazend

pyslurm/core/reservation.pxd

@@ -72,9 +72,9 @@ cdef class Reservation:
             Name for a Reservation.
         **kwargs (Any, optional=None):
             All Attributes of a Reservation are eligible to be set, except
-            `cpus_by_node`. Although the `name` attribute can also be changed
-            on the instance, the change will not be taken into account by
-            `slurmctld` when calling `modify()`.
+            `cpu_ids_by_node`. Although the `name` attribute can also be
+            changed on the instance, the change will not be taken into account
+            by `slurmctld` when calling `modify()`.
 
     Attributes:
         accounts (list[str]):
@@ -85,7 +85,7 @@ cdef class Reservation:
             Arbitrary comment for the Reservation.
         cpus (int):
             Amount of CPUs used by the Reservation
-        cpus_by_node (dict[str, int]):
+        cpu_ids_by_node (dict[str, int]):
             A Mapping where each key is the node-name, and the values are a
             string of CPU-IDs reserved on the specific nodes.
         end_time (int):

pyslurm/core/reservation.pyx

@@ -327,7 +327,7 @@ cdef class Reservation:
         self.info.core_cnt = self.umsg.core_cnt = int(val)
 
     @property
-    def cpus_by_node(self):
+    def cpu_ids_by_node(self):
         out = {}
         for i in range(self.info.core_spec_cnt):
             node = cstr.to_unicode(self.info.core_spec[i].node_name)

tests/unit/test_reservation.py

@@ -32,7 +32,7 @@ def test_create_instance():
     assert resv.end_time == None
     assert resv.duration == 0
     assert resv.is_active is False
-    assert resv.cpus_by_node == {}
+    assert resv.cpu_ids_by_node == {}
     assert resv.to_dict()
 
     start = datetime.now()