Better support for multiple multiplexing switches

[jstroem]

Does also add support for SG_MUL_VAL_ defs

[hashemmm96]

Hey, thanks for your contribution! Sorry for the late review 🙇

I think this looks nice. However we've had some changes in the CI for this project so we do some additional checks on PRs. Let's start by rebasing this PR on latest master, then check if there are additional changes needed.

[hashemmm96]

Actually, by adding this feature, the dbc file linter will complain about multiple multiplexing switches even though they are supported. This can be remediated by modifying the linter in pkg/dbc/analysis/passes/multiplexedsignals/analyzer.go

[tomrijntjes]

I would appreciate this feature very much. Is there anything I can do?

edit: merging main works fine without conflicts. I'm not sure how to reproduce the linter issue mentioned above.

[bryanjhv]

@tomrijntjes something like this.

go run go.einride.tech/can/cmd/cantool lint <file-or-dir>

[RangelReale]

In the python cantools tests, there is a dbc file with a list of multiplex ranges in the SG_MUL_VAL_ message.
Is there a documentation of this format?

[Jassob]

Hello @RangelReale, we've used http://www.mcu.so/Microcontroller/Automotive/dbc-file-format-documentation_compress.pdf as reference.

You can cross-reference it for information about SG_MUL_VAL_.

[Jassob]

@jstroem Please confirm if you're still interested in getting this PR merged, otherwise we'll close it in 14 days unless someone adopts it until then.

[Jassob]

Please rebase on current master