Skip to content

New merge of rileys discovery work - #9

Open
davidraker wants to merge 19 commits into
eclipse-volttron:developfrom
davidraker:new_merge_of_rileys_discovery_work
Open

davidraker wants to merge 19 commits into
eclipse-volttron:developfrom
davidraker:new_merge_of_rileys_discovery_work

Conversation

@davidraker

Copy link
Copy Markdown
Contributor

This pull request adds a new BACnet network scanning tool, updates dependencies, and improves compatibility and usability for BACnet device discovery. The most significant changes include the introduction of a modernized bacnet-scan script using BACpypes3 and asyncio, enhancements to dependency management, and improved configuration flexibility.

New BACnet network scanning tool:

  • Added a new script src/protocol_proxy/protocol/bacnet/tools/scan.py that performs WHO-IS discovery of BACnet devices using BACpypes3 and asyncio. This tool is backward compatible with the legacy script, supports both broadcast and unicast scans, allows exporting results to CSV, and provides flexible configuration via INI files or command-line arguments.

Dependency and packaging improvements:

  • Added psutil as a required dependency and registered two new CLI entry points (bacnet-scan and bacnet-grab-config) in pyproject.toml for easier tool invocation.
  • Updated the Python version requirement for mypy and Poetry to "python_version = \">=3.10,<4.0\"" for better compatibility.

Configuration and documentation updates:

  • Fixed the installation instruction in README.md to use the correct package name (protocol-proxy-bacnet).

BACnet protocol bugfix:

  • Fixed a bug in bacnet.py where the standard BACpypes3 who_is call incorrectly used the broadcast address instead of None for broadcast, improving scan reliability.

@davidraker
davidraker requested a review from schandrika May 5, 2026 17:46
@riley206-pnnl

Copy link
Copy Markdown
Contributor

Looks good to me.

@shwethanidd
shwethanidd self-requested a review July 29, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants