Closed gcoan closed 2 months ago
If I turn on debug mode I am not seeing any difference in the original predbat plan column descriptions. What else do I need to do to trigger this outcome in the original plan exactly?
You need to be using the predbat version that’s on main, not the public release but it’ll be doubtless published soon by Trefor On 23 Apr 2024 at 14:00 +0100, pacemaker82 @.***>, wrote:
If I turn on debug mode I am not seeing any difference in the original predbat plan column descriptions. What else do I need to do to trigger this outcome in the original plan exactly? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>
so is it using these truncated status's only if the new CO2 columns is added, or all the time now?
could you try this? https://github.com/pacemaker82/PredBat-Table-Card/releases/tag/v1.5.1.10
Trefor has added/is adding new capability to plan charging when "greenest" https://github.com/springfall2008/batpred/issues/980
Needs I think the National Grid integration (which I don't have at present), and then its another switch to enable it.
If enabled then you get two extra columns in the html plan from what I've seen of the code.
But looks like with the extra columns Trefor has decided to shorten the statii labels to ensure the plan fits onto a page. This change to the labels is permanent regardless of whether those columns are shown or not.
I'm trying the version on main to see if it improves a 'stuck in hold charge' I experienced last night and noticed that the labels had changed today.
Trying the 1.15.1.10, it displays charge and hold charge OK
ok well it looks like my fix fixed it anyway and is backwards compatible too.
Fixed for Discharge as well
Don't seem to be able to reopen this
Just noticed force discharge is truncated
@gcoan should be fixed in https://github.com/pacemaker82/PredBat-Table-Card/releases/tag/v1.5.2.1
@pacemaker82 old text:
Which I definitely didn't like.
New text:
I'm not massively loving the length of the new text, worried it'll scroll the plan unnecessarily, but could maybe live with it. FreezeChrg now looks odd in comparison, as 'Freeze Charge' is no longer than 'Force Discharge'
'Freeze Discharge' in full is of course getting even longer ...
screenshots are the same, can you provide the comparison image so I have context?
Doh!
Here's the new screen
is the best or expected option to show "Force Chrg" and "Force Dischrg" ?
I don't know. It's a trade off of space vs readability
How about
Freeze Charge Freeze Dischrg Force Charge Force Dischrg
?
Looking again
FreezeChrg looks OK and doesn't push the width too much but both the Force's do make the column wider.
Of course Freeze Discharge is going to be the widest possible one. So maybe find the shortest acceptable version of that and work backwards
I'm going to adjust to "Force Dischrg" and leave the rest
this looks a lot better than the standard Predbat statii labels that are too truncated now in my view
Could split FreezeChrg into two words but then that'd be inconsistent with FreezeDischrg which is getting too long anyway
Thanks
Despite being in landscape on my ipad I am seeing the Charge/Discharge statii truncated today.
Looks like predbat html plan is truncating the text which must be a new thing in the main() branch. Can the table card revert this?
Previous predbat version:
Looking at the code for the main branch, there are new columns added for CO2, some of the labels change in debug mode, and several of the status text labels have been truncated: