Skip to content
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

Added in patch to support PnC #30

Closed

Conversation

louisg1337
Copy link

Describe your changes

Hello everyone, I am apart of the team over at NREL who has been working on the SIL demos in everest/everest-demo. Recently, @shankari went to the CharIN Testival and showcased EVerest by running a custom demo that was tailored to have PnC functioning, found here. To get PnC to work, she had to apply a handful of patches to different parts of EVerest, all which can be found here. We now would like to slowly integrate these patches into EVerest so that it can fully support PnC.

This patch allows us to cycle between using PnC and EIM authentication methods by what is chosen in the EVerest simulator.

Testing Done

As mentioned above, this patch is one of many, so it has only been tested in conjunction with the other patches in the SIL demo. The particular demo which has PnC working can be found linked above. To run this demo, I would recommend...

  1. Clone repo and check out branch.
    git clone https://github.com/US-JOET/everest-demo.git
    git checkout -b test-demo origin/charin-e2e-demo  
    
  2. Run the software simulation demo.
    bash demo-iso15118-2-ac-plus-ocpp.sh -r $(pwd) -b test-demo -3   
    
  3. Wait until the EVerest software boots up and you see the following.
    2024-06-18 18:23:22.836788 [INFO] ocpp:OCPP201     :: Received BootNotificationResponse: {
        "currentTime": "2024-06-18T18:23:22.000Z",
        "interval": 300,
        "status": "Accepted"
    }
    
  4. Navigate to http://localhost:1880/ui/, choose from the Car Simulation dropdown menu AC ISO15118-2 Plug&Charge, then plug the car in and observe it authenticate on its own.

Issue ticket number and link

The issue can be found here in everest-demo.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@SebaLukas
Copy link
Collaborator

SebaLukas commented Jun 27, 2024

This PR will not be accepted and will be closed in a few days.
See matching PR in everest-core for the next steps

@SebaLukas SebaLukas self-assigned this Jul 2, 2024
@SebaLukas
Copy link
Collaborator

So I close this one. See matching PR in everest-core.

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.

2 participants