-
Notifications
You must be signed in to change notification settings - Fork 64
EVI-EnLitePy Endpoint for EV Charging Profiles #660
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
Conversation
Kept Django at LTS version 4.2 for compatibility Kept numpy at v1 due to pyeasiur -> deepdish pkg incompat with numpy v2
…ith enrichment_version=1; update UI input contract docs; comparison script defaults to enriched output
|
@Bill-Becker This sets default enrichment (normalized + annual metrics + utilization) and adds enrichment_version=1 so the UI only sends the minimal input JSON (chargers / vehicles / arrival / maxChargeDurationMin). run_enlitepy_compare.py defaults were updated to match. Docs now clearly explain the simplified contract. No breaking changes—only additive. We can remove the helper scripts once we have a working test with the ui |
run_enlitepy_compare.py
Outdated
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.
@bpulluta we'll want to delete this before merging if it's just for reference to migrate code to the /ensite view.
README_ENSITE_UI.md
Outdated
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.
@bpulluta should this go inside of the /load_builder directory, or did you want to keep it at the top-level?
|
@bpulluta I assume you tested this locally based on the documentation, but just checking if you wanted me to deploy this update to the development server yet? |
@Bill-Becker I couldn't run the docker container locally to test but i did test with the deployed server so i think this will work. Can you test locally and confirm that it works. We can remove the extra script and README that i used for testing |
I think the README is useful to keep if we think it will help users/developers, but maybe we move it into that /load_builder folder where the /ensite endpoint lives. |
…max grid utilization metric
…max grid utilization
replicating the working migrations from `develop` and creating migration file for any missing model inputs or outputs
rathod-b
left a comment
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.
Validated expected API functionality and fixed some migration issues.
|
@Bill-Becker @rathod-b, everything seems to be working on my side @Bill-Becker will you be merging? |
bpulluta
left a comment
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.
I tested Ensite intergation and it works correctly
Uh oh!
There was an error while loading. Please reload this page.