-
Notifications
You must be signed in to change notification settings - Fork 9
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
Web App for ILC Config file generator #38
Comments
@bbartling would you be willing to create a PR with changes that have helped you in your usage? I think that would be awesome! |
Yes absolutely. Should I fork a copy of this repo? |
Yes! :) |
Working with @rlutes today here are some notes about issues we ran into on the web app to help generate ILC config.
Add in some logic to the web app we dont need
I have to manually ADD in:
|
Some more items to add to the deficiencies that @rlutes and I noticed when using the web app to create the ILC config file:
Typo :
In the config that states keys
They are backwards, they need to be:
More typos in the control config device status keywords incorrect
|
In the main ILC configuration config, the parameter curtailment_confirm should be changed to confirm_time. Also, curtailment_break is no longer used in the code, remove from configuration. |
@bbartling - Replying to this, is the intent to copy device topic info from one device to multiple? This can be done by performing the copy on all devices wanting to have the values from another device. Please let me know what's desired here. Not exactly sure what you're trying to do. |
Is it possible to add more documentation to the ILC README and website to configure the ILC agent for how to get the Master Driver Configuration Store file?
I think the average slow to catch on types like maybe myself this is blatantly not obvious for how to get this file. At least I dont think there is anything I could find in the docs for VOLTTRON and no one else ever asked this on stackoverflow.
Thanks @rlutes for the stackoverflow answer, step one is checked off the list : )
The text was updated successfully, but these errors were encountered: