Skip to content

feat: support .blueprintignore#365

Open
kulti wants to merge 2 commits into
mainfrom
abakin/bp-ignore
Open

feat: support .blueprintignore#365
kulti wants to merge 2 commits into
mainfrom
abakin/bp-ignore

Conversation

@kulti
Copy link
Copy Markdown
Contributor

@kulti kulti commented Apr 27, 2026

No description provided.

@kulti kulti force-pushed the abakin/bp-ignore branch 4 times, most recently from 0609150 to a60a8e9 Compare April 28, 2026 13:26
@kulti kulti force-pushed the abakin/bp-ignore branch from a60a8e9 to 0afa7dc Compare April 28, 2026 14:28
@kulti kulti force-pushed the abakin/bp-ignore branch from 0afa7dc to 5385946 Compare April 28, 2026 14:55
Copy link
Copy Markdown
Contributor Author

@kulti kulti left a comment

Choose a reason for hiding this comment

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

Status mapping need to review.

elseif value == 'fault' then
return 'fault'
else
enapter.log('unmapped vendor status: ' .. tostring(value), 'warn')
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't know how to map standalone status. Claude made a suggestion, that is operating, but I'm not sure and cannot find any description of this status.

elseif current < 0 then
return 'discharging'
else
return 'idle'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

How to implement float battery charge status? I have an idea to set float if charging and abs(charging) < float_threshold.

other:
display_name: Other
description: Other battery type, not covered by the standard types
battery_nominal_voltage:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Deye Inverter has voltage between 40-60 V. I don't understand, how to set this properties. If it change dynamically, we probably should average value 50V?

@kulti kulti requested a review from nikitug April 29, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant