<div dir="ltr"><div dir="ltr">Actually, I do have one thought.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 8, 2024 at 4:45 PM Nicholas Pratte <<a href="mailto:npratte@iol.unh.edu">npratte@iol.unh.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">+    def set_mac_addr(self, port_id: int, mac_address: str, add: bool, verify: bool = True) -> None:<br>
+        """Add or remove a mac address on a given port's Allowlist.<br>
+<br>
+        Args:<br>
+            port_id: The port ID the mac address is set on.<br>
+            mac_address: The mac address to be added to or removed from the specified port.<br>
+            add: If :data:`True`, add the specified mac address. If :data:`False`, remove specified<br>
+                mac address.<br></blockquote><div><br></div><div>Should we replace "add" with "set" or an equivalent? Add=True means add. Add=false doesn't necessarily mean remove... set=True and set=False reads a little clearer for a function where the boolean is adding/removing something. But, it's just a thought - add is probably fine if you think that that's what reads best. </div><div> </div></div></div>