Skip to content

Commit

Permalink
Added checklist for new BMS
Browse files Browse the repository at this point in the history
  • Loading branch information
mr-manuel committed May 22, 2023
1 parent fe2b5f3 commit 7c32cfc
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/docs/general/features.md
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ CCCM limits the charge/discharge current depending on the highest/lowest cell vo
* between `2.8V - 2.9V` → `5A `discharge
* below `<= 2.70V` &rarr; `0A` discharge

### Temprature
### Temperature

* `CCCM_T_ENABLE = True/False`
* `DCCM_T_ENABLE = True/False`
Expand Down
17 changes: 15 additions & 2 deletions docs/docs/general/supported-bms.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,22 @@ You can view the current [BMS requests](https://github.com/Louisvdw/dbus-serialb
## Add/Request new BMS
There are two possibilities to add a new BMS.

1. Fork the repository and use the [`battery_template.py`](https://github.com/mr-manuel/venus-os_dbus-serialbattery/blob/master/etc/dbus-serialbattery/bms/battery_template.py) as template to add a new battery. As soon as the BMS works you can open a PR (pull request) to merge it.
### Add by opening a pull request
Fork the repository and use the [`battery_template.py`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/etc/dbus-serialbattery/bms/battery_template.py) as template to add a new battery. As soon as the BMS works you can open a pull request (PR) to merge it. Please use the `dev` branch for adding your BMS/feature and opening the PR.

2. Start a [new discussion](https://github.com/Louisvdw/dbus-serialbattery/discussions/new?category=new-bms-requests) in the `New BMS request` category. Please add also the protocol documentation which you can request from the manufacturer/seller. The more upvotes the BMS request has, the higher is the priority.
Here is a short checklist that should help you:

- [ ] Add your battery class and battery class import in alphabetical order in the [`etc/dbus-serialbattery/battery.py`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/etc/dbus-serialbattery/battery.py)
- [ ] Add your BMS to the [BMS feature comparison](https://louisvdw.github.io/dbus-serialbattery/general/features#bms-feature-comparison) page by editing [`docs/docs/general/features.md`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/docs/docs/general/features.md)
- [ ] Add your BMS to the [Supported BMS](https://louisvdw.github.io/dbus-serialbattery/general/supported-bms) page by editing [`docs/docs/general/supported-bms.md`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/docs/docs/general/supported-bms.md)
- [ ] Do not import wildcards `*`
- [ ] If your BMS don't run with the default settings add installation notes to the [How to install, update, disable, enable and uninstall](https://louisvdw.github.io/dbus-serialbattery/general/install#bms-specific-settings) [`docs/docs/general/install.md`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/docs/docs/general/install.md)
- [ ] If your BMS needs custom settings that the user should be able to change, add it below the `; --------- BMS specific settings ---------` section in the [`etc/dbus-serialbattery/config.default.ini`](https://github.com/Louisvdw/dbus-serialbattery/blob/dev/etc/dbus-serialbattery/config.default.ini)
- [ ] Make sure the GitHub Actions run fine in your repository. In order to make the GitHub Actions run please select in your repository settings under `Actions` -> `General` -> `Actions permissions` the option `Allow all actions and reusable workflows`. Check also in your repository settings under `Actions` -> `General` -> `Workflow permissions` if `Read and write permissions` are selected. This will check your code for Flake8 and Black Lint errors. [Here](https://py-vscode.readthedocs.io/en/latest/files/linting.html) is a short instruction on how to set up Flake8 and Black Lint checks in VS Code. This will save you a lot of time.


### Request by opening a discussion
Start a [new discussion](https://github.com/Louisvdw/dbus-serialbattery/discussions/new?category=new-bms-requests) in the `New BMS request` category. Please add also the protocol documentation which you can request from the manufacturer/seller. The more upvotes the BMS request has, the higher is the priority.

If you would like to donate hardware or would like to help testing a specific BMS please get in contact over the [discussions section](https://github.com/Louisvdw/dbus-serialbattery/discussions).

Expand Down

0 comments on commit 7c32cfc

Please sign in to comment.