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

WIP emhass csv append modules and fix inverters syntax and semantics #426

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

GeoDerp
Copy link
Contributor

@GeoDerp GeoDerp commented Jan 7, 2025

With Raf's help, we have spotted a few issues with the custom inverters. pvlib seems very partiular with what data it would like, so more testing is needed

@GeoDerp
Copy link
Contributor Author

GeoDerp commented Jan 18, 2025

@davidusb-geek, the last commit 8106fea is a quick proof of concept then anything. It adds def_total_timestep parameter (runtime only for now) that can be used instead of def_total_hours for passing timestep number instead of hour number. The hope is to fix/help this issue: #373

Any ideas on this would be appreciated.

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.

1 participant