Nathan Hansen

I think I've addressed the other issues, but am still getting image comparison failures. Is this to be expected when making changes related to how plots are displayed?

> If you please can provide the resulting figures from e.g. #22861 after applying the changes here, it would simplify the review. I've attached some screenshots below showing the updated...

I feel like I should explain the comment "# Solid near, transparent far, solid default" further: depending on the equation used for calculating "sats" you can change the behavior of...

> * can we use an enum (or strings) rather than integers for the modes? Definitely, though I'm not sure what appropriate string representations would be. My previous convention was...

> Hi @nhansendev, are you still interested in finishing up this bugfix/feature? Thanks! Thanks for the reminder, I've made some updates. I've thought about the changes I had proposed and...

FYI, the methods using `_get_transport()` didn't work for me, but I was able to use `socket.setdefaulttimeout` instead: ``` import socket from paramiko import Transport # Must be called before the...

Based on other, similar issues I think the problem is that not all tensors are being sent to the GPU when Cuda is available. I'm trying to find where ".to(self.device)"...

Added a pull request to address the issue by adding a config_file option. The config file can contain paths to the identity and known hosts files, so it solves the...

This error is appears when the example code is not run in a jupyter Notebook. It is implied, but not explicitly stated that the code should be run in a...

I figured out what I was doing wrong. 1. Use serv.serve_forever() instead of serv.start() 2. Run it in a thread 3. Connect the socket on the serv.listen_port instead Working class...