Skip to content

Update dallas_temp.md #5847

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
itofficeeu wants to merge 1 commit into
base: current
Choose a base branch
from
Draft

Update dallas_temp.md #5847

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions content/components/sensor/dallas_temp.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,11 @@

- All other options from [Sensor](/components/sensor).

### Byte Order of the Address

Check failure on line 49 in content/components/sensor/dallas_temp.md

View workflow job for this annotation

GitHub Actions / Lint 

Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Byte Order of the Address"]
Copy link
Contributor

@coderabbitai coderabbitai bot Jan 3, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Add a blank line before the heading.

The linting check requires headings to be surrounded by blank lines. Add a blank line between line 48 and the heading.

🔎 Proposed fix 
  - All other options from [Sensor](/components/sensor).

~
### Byte Order of the Address
🧰 Tools
🪛 GitHub Check: Lint

[failure] 49-49:
Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Byte Order of the Address"]

🤖 Prompt for AI Agents 
In content/components/sensor/dallas_temp.md around line 49, the heading "###
Byte Order of the Address" is not preceded by a blank line; add a single blank
line between line 48 and this heading so the heading is surrounded by blank
lines to satisfy the lint rule.

Copy link
Contributor

@ximex ximex Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls fix

Copy link
Contributor

@coderabbitai coderabbitai bot Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ximex, I'll fix the linting issue by adding a blank line after the heading.

Task: Fix markdown linting issue

  • Add a blank line after the "### Byte Order of the Address" heading
  • Remove trailing space from the heading

💡 For code changes across multiple files, use @coderabbitai plan in an issue comment

EspHome (most recent version 2026-January) - uses reversed order of the bytes, compared to the Arduino IDE. That means, if you have an address for a sensor via the Arduino IDE that says 0x2870F5E0161301C7 - then you have to use 0xc7011316e0f57028 in EspHome. The bytes are pairwise ordered in reverse. The Arduino IDE version ends in the above example with c7 - the EspHome version the same address starts with c7. - The other pairs accordingly. The arduino IDE is LSB‑first. While EspHome is MSB‑first.
Copy link
Contributor

@ximex ximex Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
EspHome (most recent version 2026-January) - uses reversed order of the bytes, compared to the Arduino IDE. That means, if you have an address for a sensor via the Arduino IDE that says 0x2870F5E0161301C7 - then you have to use 0xc7011316e0f57028 in EspHome. The bytes are pairwise ordered in reverse. The Arduino IDE version ends in the above example with c7 - the EspHome version the same address starts with c7. - The other pairs accordingly. The arduino IDE is LSB‑first. While EspHome is MSB‑first.
ESPHome starting with 2026.1.0 uses reversed order of the bytes, compared to the Arduino IDE. That means, if you have an address for a sensor via the Arduino IDE that says `0x2870f5e0161301c7`, then use `0xc7011316e0f57028` in ESPHome. The bytes are pairwise ordered in reverse. The Arduino IDE version ends in the above example with `c7`, the ESPHome version the same address starts with `c7`. The other pairs accordingly. The Arduino IDE is LSB‑first, while ESPHome is MSB‑first.

Copy link
Member

@ssieb ssieb Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there an esphome version mentioned at all? It has always done it this way.


If you don't have the addresses, use index instead of addresses for the first run. EspHome will print the addresses on the log at first use with index. After that, try with addresses, if you want.
Copy link
Contributor

@ximex ximex Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If you don't have the addresses, use index instead of addresses for the first run. EspHome will print the addresses on the log at first use with index. After that, try with addresses, if you want.
If you don't have the addresses, use index instead of addresses for the first run. ESPHome will print the addresses on the log at first use with index. If you have the addresses its recommended to use a configuration with them.

Copy link
Member

@ssieb ssieb Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you don't have the addresses, then just run it without any sensors configured. It will always print the detected addresses in the log.


### See Also

- [Arduino DallasTemperature library](https://github.com/milesburton/Arduino-Temperature-Control-Library)
Expand Down
Loading