Create external csms demo #109
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
update to demo-iso15118-2-ocpp-201.sh from deprecated demo-iso15118-2-ac-plus-ocpp.sh Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
AC -> DC Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
It looks like the limits applied to the JsYetiSimulator are on a per-phase basis, although limits applied elsewhere are not. This makes things very confusing. We will fix this by configuring everything to a per-phase limit of 16A Please see EVerest#92 (comment) to EVerest#92 (comment) Locations changed: - config - JsYetiSimulator - SmartCharging OCPP defaults Also fix the disable_iso_tls patch to not have a starting `/` Add a new patch file to enable limit logging but don't enable it by default. This may help future efforts to debug this. Signed-off-by: Shankari <k.shankari@driveelectric.gov>
Consistent with EVerest#92 (comment) History starts at: EVerest#92 (comment) Also show dots in the chart Also convert the time range to/from local time so it makes sense Signed-off-by: Shankari <k.shankari@driveelectric.gov>
Signed-off-by: Shankari <k.shankari@driveelectric.gov>
🔧 configure limits in a way that is consistent and makes sense
- Do not throttle the SASchedule from the EV based on departure time, only transmit the composite pmax schedule. The EV will deareate based on departure time - Show the values in the current power delivery request instead of the progress towards the eamount so that we can see the impact of curtailing the pmax - Pass in the pmax to the power curve computation algo, although it is currently a NOP This fixes: EVerest#92 (comment) Signed-off-by: Shankari <k.shankari@driveelectric.gov>
+ configure the chart to display kW + convert relative timestamps to display timestamps by adding to the start time of the charge session Signed-off-by: Shankari <k.shankari@driveelectric.gov>
Signed-off-by: Shankari <k.shankari@driveelectric.gov>
Make the EV-computed curve more meaningful
Updates to ReadME for 1-line demos
…hanges Changes: - EVerest manager uses the changes to receive a limit through the API, pass it through to the OCPP module, store it in the database and use it to influence the power limits - Node-red sends the correct MQTT messages - MaEVe manager has an NOP implementation of notify limit The actual changes/patches are listed here and will need to be merged in one step at a time EVerest#101 (comment) Testing done: ``` $ bash demo-iso15118-2-ocpp-201.sh -1 -r $(pwd) -m $ docker exec -it everest-ac-demo-manager-1 /bin/bas (container) $ sh /ext/build/run-scripts/run-sil-ocpp201-pnc.sh ``` Then - moved the slider to various points, the max current changed - plugged in car, handshake was successful - moved the slider while the car was plugged in, max current changed and power drawn changed - cleared limit, max current went to 16 and max power went to 11kW Signed-off-by: Shankari <k.shankari@driveelectric.gov>
💩 🔧 Configure the demo to use pre-built images for the notify limit c…
required for demo functionality after switching to one phase so charging will actually initiate Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
…onfig add three_phases to JsEVManager
merge upstream
demo follows demo-iso15118-2-ocpp-201.sh except: checks for url in environment, if not configured, exits tested by: setting url as patch would, working normally, removed, failed with error message Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
include instructions to set csms url Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
clarify direction and give and example Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
make it more clear what is code, and what is prose Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
Collaborator
Author
|
Testing done: had regular 1-line demo up, ran before setting url then after saw log error before setting, after setting seemed to work. Able to run manager and charge in UI just as expected! |
Collaborator
Author
|
Wanted to test with something else ... maeve requires me to work out the certificate situation ... I don't think I can chase that down tonight, but I am reasonably confident that this script does what we think it does because I was able to set the url to the working maeve container and see an error when it was not set - so the environment variable does what we want it to do. If I manage to find some time in the morning I'll work through as much of the certificates as I can. |
Collaborator
Author
|
maeve failed: so did citrene: |
Collaborator
Author
|
Quick test w/ Cintrene by calling our docker compose, then exporting the same url as the patch: Getting issues with the nodered container in EVerest which won't start, so wiping and trying again |
Collaborator
Author
|
Error: It looks like citrene and everest are trying to use the same port? In any case, this is looking like a fail for now, running the manager and OCPP is not connecting to the client: |
Collaborator
Author
|
Trying MobilityHouse with the EVerest implementation (https://github.com/EVerest/ocpp-csms) and it is also erroring out, complaining about some dependencies: |
Collaborator
Author
|
This is because ocpp (directly from MobilityHouse) is a requirement of the EVerest scripts, but not by a tagged version ... so they're out of sync |
Signed-off-by: Abby Wheelis <54848919+Abby-Wheelis@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In alignment with #101 (comment), creating a quick script to highlight to CSMS's that they can test with EVerest!