BMS DBC files and BMS dashboard by Cdella778 · Pull Request #168 · UBC-Solar/sunlink

Cdella778 · 2026-07-04T23:31:40Z

Sunlink Pull Request Description

Monday Link

Effected Components

Testing

Vehicle testing
Randomizer testing
Other
N/A

Sanity check

CAN ID table / DBC updated
gitignore updated and commited
Steps confirmed

Sources

AarjavJain101

Needs changes

Lookign for changes.

AarjavJain101

Changes needed.

AarjavJain101 · 2026-07-06T01:29:19Z

+
+BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST
+
+BO_ 770 HvcHeartbeat: 2 HVC


Upper case HVC and MST in the names. Makes it easy to read. They are acronyms after all.

AarjavJain101 · 2026-07-06T01:29:24Z

+
+BS_:
+
+BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST


AarjavJain101 · 2026-07-06T01:29:42Z

+
+BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST
+
+BO_ 770 HvcHeartbeat: 2 HVC


Follow same naming conventions as all the other messages. That is TimeSinceBootUp.

AarjavJain101 · 2026-07-06T01:31:10Z

+BO_ 773 HvcShuntCurrent: 4 HVC
+ SG_ ShuntCurrent : 7|32@0- (1,0) [-2147483648|2147483647] "mA" Vector__XXX


Name the signal name to "PackCurrent". The message name can stay the same. This is because when STG or EMD are looking at the data and they see ShuntCurrent its not clear what that means. We have used Pack Current in the past as well so that familiarity sticks.

AarjavJain101 · 2026-07-06T01:31:17Z

+BO_ 774 DistHeartBeat: 3 DIST
+ SG_ DistHeartbeat : 0|1@1+ (1,0) [0|1] "" Vector__XXX


Use All caps for DIST.

AarjavJain101 · 2026-07-06T01:31:31Z

+ SG_ ScrutineeringEnable : 45|1@1+ (1,0) [0|1] "" Vector__XXX
+BO_ 785 VoltSummary: 6 MST


Add a new line. Keep the same formatting as what was in the file before.

AarjavJain101 · 2026-07-06T01:32:18Z

+ SG_ Mod11Status m2 : 32|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod12Status m3 : 8|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod13Status m3 : 16|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod14Status m3 : 24|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod15Status m3 : 32|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod16Status m4 : 8|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod17Status m4 : 16|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod18Status m4 : 24|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod19Status m4 : 32|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod20Status m5 : 8|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod21Status m5 : 16|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod22Status m5 : 24|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod23Status m5 : 32|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod24Status m6 : 8|8@1+ (1,0) [0|255] "" Vector__XXX
+ SG_ Mod25Status m6 : 16|8@1+ (1,0) [0|255] "" Vector__XXX


Can you confirm that this DBC file is what we used during our Saturday integration test? I want to make sure that the multiplexing is working.

Can you also give a brief reply as to why the voltage reading's were inaccurate on Grafana on Saturday? Is the fix for that in the DBC or in FW? This dictates whether this PR needs more changing or not.

AarjavJain101 · 2026-07-06T01:32:48Z

+BO_ 787 ModuleVoltData: 5 MST
+ SG_ VoltGroupIdx M : 0|4@1+ (1,0) [0|15] "" Vector__XXX


Voltage is more clear and easier read to me.

AarjavJain101 · 2026-07-06T01:33:06Z

+ SG_ Mod31Temp m7 : 32|8@1- (1,0) [-127|127] "degC" Vector__XXX
+BO_ 789 ModuleStatus: 8 MST
+ SG_ StatusGroupIdx M : 0|3@1+ (1,0) [0|7] "" Vector__XXX
+ SG_ Mod0Status m0 : 8|8@1+ (1,0) [0|255] "" Vector__XXX


Status of what about a module? This name is a bit ambiguous.

AarjavJain101

A few more changes.

AarjavJain101 · 2026-07-06T01:38:57Z

+BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST
+
+BO_ 770 HvcHeartbeat: 2 HVC
+ SG_ HvcHeartbeatCounter : 7|16@0+ (1,0) [0|65535] "" Vector__XXX


Make this 32 bits. Standardized. All TimeSinceBootUp messages should follow the same standard for ease and organization in the future.

AarjavJain101 · 2026-07-06T01:42:59Z

+ SG_ ShuntCurrent : 7|32@0- (1,0) [-2147483648|2147483647] "mA" Vector__XXX
+
+BO_ 774 DistHeartBeat: 3 DIST
+ SG_ DistHeartbeat : 0|1@1+ (1,0) [0|1] "" Vector__XXX


This should also be 32 bits. A heartbeat is a counter.

AarjavJain101 · 2026-07-06T01:44:00Z

+ SG_ SpareCurrent : 32|8@1+ (5,0) [0|1275] "mA" Vector__XXX
+
+BO_ 784 BmsStatus: 8 MST
+ SG_ BmsHeartbeatCounter : 0|32@1+ (1,0) [0|4294967295] "" Vector__XXX


MST heartbeat counter. BMSStatus is ok because it is the status of our BMS.

AarjavJain101 · 2026-07-06T04:27:22Z

+  },
+  "time": {
+    "from": "now-5m",
+    "to": "now"


Change to 'now-1m' so your dashboard always starts with the 1 minute window option which is ideal for our sampling speed. 5 minutes is too long and Grafana cannot show all the datapoints in a 5 minute window (you will see a red X in a triangle in the corner of your panels as a result).

AarjavJain101 · 2026-07-06T04:31:53Z

+    "from": "now-5m",
+    "to": "now"
+  },
+  "timepicker": {},


For timepicker look at what the other dashboards have and copy that. You need to have a "1s" option to refresh Grafana at 1s. I recommend you try "500ms, for example to see if it lets you (im unsure about this).

AarjavJain101 · 2026-07-06T04:32:54Z

+  },
+  "timepicker": {},
+  "timezone": "",
+  "title": "BMS Dashboard",


Change this title to follow the standard outlined in GRAFANA.md.

BMS DBC files and BMS dashboard

772deb7

Cdella778 requested a review from AarjavJain101 July 4, 2026 23:31

AarjavJain101 previously approved these changes Jul 6, 2026

View reviewed changes

Comment thread dbc/brightside.dbc Outdated

AarjavJain101 self-requested a review July 6, 2026 01:20

Updated all Brightside refs to Cascadia

e5a3941

AarjavJain101 requested changes Jul 6, 2026

View reviewed changes


		BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST

		BO_ 770 HvcHeartbeat: 2 HVC


		BS_:

		BU_: AMB BMS ECU TEL OBC MEMORATOR MDI MC STR DRD MPPT DIST MST

		BO_ 773 HvcShuntCurrent: 4 HVC
		SG_ ShuntCurrent : 7\|32@0- (1,0) [-2147483648\|2147483647] "mA" Vector__XXX

		BO_ 774 DistHeartBeat: 3 DIST
		SG_ DistHeartbeat : 0\|1@1+ (1,0) [0\|1] "" Vector__XXX

		SG_ ScrutineeringEnable : 45\|1@1+ (1,0) [0\|1] "" Vector__XXX
		BO_ 785 VoltSummary: 6 MST

		BO_ 787 ModuleVoltData: 5 MST
		SG_ VoltGroupIdx M : 0\|4@1+ (1,0) [0\|15] "" Vector__XXX

Uh oh!

Conversation

Cdella778 commented Jul 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Sunlink Pull Request Description

Monday Link

Effected Components

Testing

Sanity check

Sources

Uh oh!

AarjavJain101 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

AarjavJain101 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AarjavJain101 Jul 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AarjavJain101 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Cdella778 commented Jul 4, 2026 •

edited

Loading

AarjavJain101 Jul 6, 2026 •

edited

Loading