<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote gmail_quote_container"><div>The reason for the relocation of these methods is not entirely <br>clear given the commit message. It would be helpful to give a<br>brief explanation in the commit message.<br><br> <span style="color:rgb(80,0,80)">> +    def setup(self, topology: Topology) -> None:</span></div><span style="color:rgb(80,0,80)">> +        """Overrides :meth:`.traffic_generator.</span><span style="color:rgb(80,0,80)">TrafficGenerator.setup`."""</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">> +        for port in self._tg_node.ports:</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">> +            self._tg_node.main_session.</span><span style="color:rgb(80,0,80)">configure_port_mtu(2000, port)</span><br style="color:rgb(80,0,80)"><br>Reiterating Luca's suggestion, it is cleaner to do:<br>  port.configure_mtu(2000)<span style="color:rgb(80,0,80)"><br><br></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="color:rgb(80,0,80)">@dataclass(slots=True)</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+class PerformanceTrafficStats:</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+    """Data structure to store performance statistics for a given test run.</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+    Attributes:</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+        tx_pps: Recorded tx packets per second</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+        tx_bps: Recorded tx bytes per second</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+        rx_pps: Recorded rx packets per second</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+        rx_bps: Recorded rx bytes per second</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+        frame_size: The total length of the frame</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">+    """</span>
</blockquote><div>Small nit: missing full stops on attributes. <br><br>Other than that, it looks good.</div><div><br></div><div>Reviewed-by: Andrew Bailey <<a href="mailto:abailey@iol.unh.edu">abailey@iol.unh.edu</a>></div></div></div>