Closed dhblum closed 9 months ago
Sorry that I took so long to get to this - I think it looks good! It nearly ran out of the box on my mac (there's no hostname -I on macOS so test_bacnet.py needed a 1 line change, though I just hardcoded in my IP to get past it)
I think for now just calling everything a brick:Point is fine until there's smarter semantic data that can come out of BOPTEST and actually set the point types correctly, as well as exporting data about the equipment and spaces to make the Brick model richer.
I will probably archive my repo and just point folks here.
@epaulson Thank you for reviewing! Glad it looks good. To your comments:
There's an 'ipconfig' command that works but you need to know which interface you're asking about (which is usually en0 on the mac, but not always?) The more portable option is to open a socket and then ask what IP address got attached to that socket, which will sort through whatever defaults are at play (and so should pick up the same defaults that docker using) - but as you mentioned, since that's only a CI test that will usually be on linux it's probably not a big deal.
Ok thanks @epaulson, I suggest keeping how it is then to not overcomplicate the matter.
This is ready to merge when tests pass.
This is for #547. It:
To Do:
@epaulson mind taking a review?