Interface Agnostic Handling of Multiple Interface Devices (Remove SocketCAN specific interface flag '-i') about caringcaribou

[kasperkarlsson]

See response to PR in
#53 (comment)

In general, it is a good idea to discuss things in an issue before sending a PR. I am currently not convinced that completely removing the possibility to specify an interface is the best way to handle this.

[bhass1]

Fair point about handling setups with multiple interfaces. That is not my use-case so I hadn't thought about that.

At the root of this issue is a question of how much interface configuration should be handled by Caring Caribou versus python-can. My standard procedure has been to create a ~/can.conf file per python-can recommendation and everything "just-works" with a single interface.

In the meantime, a less intrusive change would be to modify the documentation to make it more clear that -i forces SocketCAN to be used with the specified interface.

[kasperkarlsson]

Agreed. Let's keep this issue open for brainstorming/suggestions - it would be great to find an interface type agnostic way of handling this!

[kasperkarlsson]

After going through the python-can documentation, I found that the interface type specifier is now optional. Today I was (finally) able to test this out in a few different live environments, which all seem to work just fine.

@bhass1 It would be great if you could try this out too - it's in the branch https://github.com/CaringCaribou/caringcaribou/tree/interface-agnostic. The code changes were introduced in commit 38b7a8f

If no new problems are found, this will be merged to master shortly.

[Epsilon314]

When trying out with flag
-i vcan0
a NotImplementedError was raised and it seems that interface = None is passed to python-can instead of 'socketcan'.

After reading that issue, i tried to set default interface for python-can to 'socketcan' in ~/can.conf file and it works then.
Also, instead of configing python-can from file, changing line 216 in can_actions.py to
self.bus = can.Bus(DEFAULT_INTERFACE,bustype="socketcan")
works either.

Is it true that the second parameter in original
self.bus = can.Bus(DEFAULT_INTERFACE,"socketcan")
is not parsed and python-can actually uses file config?

[kasperkarlsson]

@Epsilon314 Yes. When bustype wasn't present in kwargs, older versions of python-can would fallback to reading the configuration file. In more recent versions, I believe the same thing is controlled through the channel kwarg for can.interface.Bus.

I am closing this issue now - feel free to open a new one if you encounter some unexpected behavior.