-
Notifications
You must be signed in to change notification settings - Fork 48
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.
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
Feature for SmartCharging Offline Behavior #783
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few questions and suggestions.
looks okay to me.
config/v201/component_config/standardized/SmartChargingCtrlr.json
Outdated
Show resolved
Hide resolved
"IgnoredProfilePurposesOffline": { | ||
"$comment": "Allows configuration of comma seperated list of ChargingProfilePurposes that are ignored in the composite schedule caluclation when offline.", | ||
"type": "string", | ||
"readOnly": true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the CSMS be allowed to update this?
this->device_model->get_optional_value<std::string>(ControllerComponentVariables::IgnoredProfilePurposesOffline) | ||
.value_or(""), | ||
this->is_offline()); | ||
return this->get_composite_schedule_internal(request, purposes_to_ignore); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there an EVerest style guide covering the use of this->
?
it is often needed in lambdas but here it shouldn't be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no such style guide (yet). Im coming from java (where this
is required) and personally like it to explicitly mark the members.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in a previous role member vars were prefixed with m_
Thank you for confirming there is no style guild. I have a PR where I was being asked to add this->
which at the moment is up to developer discretion.
…le in case the charging station is offline when the composite schedules are calculated by requests of a libocpp consumer application Signed-off-by: pietfried <[email protected]>
Signed-off-by: pietfried <[email protected]>
* Made IgnoredProfilePurposesOffline ReadWrite * In OCPP1.6 calling signal_set_charging_profiles_callback also in case of websocket connect or disconnect in case IgnoredProfilePurposesOffline are empty, since this can influence the composite schedules * Fixed r/w of IgnoredProfilePurposesOffline in charge_point_configuration
3e0a300
to
e977303
Compare
Signed-off-by: pietfried <[email protected]>
Describe your changes
Added configuration variable to 1.6 and 2.0.1 to allow ignoring profile in case the charging station is offline when the composite schedules are calculated by requests of a libocpp consumer application
Issue ticket number and link
#781
Checklist before requesting a review