-
Notifications
You must be signed in to change notification settings - Fork 11
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
DC transmission line feature #5
Comments
Initial code modifications are here: I am working on dclinecost and adding a test. Comments are welcome. |
My original concern was related to the fact that
Thinking out load here ... If I recall, there were a few reasons we did not include calls to
So, because of 2 you'll want to make sure that all generators that you want to be considered are on-line in the base case. Since MOST already requires consecutive numbering of buses starting at 1, the call to I'm afraid 3, the re-ordering of generators, might still be an issue with this approach, since it could potentially make the ordering inconsistent between the |
Yes, that is right. The current implementation I am working on requires that the user invokes the callbacks by hand as you described above( Thank you for pointing out the consequences of calling I assume the way I am going with this implementation is doable and I could maybe do some checks to make sure the ordering does not get messed up but it would not be a very clean implementation. |
Yes, I think fixing 1 would deal with the consequences of 3. Regarding 3, however, unfortunately it looks like |
I just pushed a commit (710277f) to the MATPOWER |
Add DC line feature to most. The following implementation is needed:
Additional the dclinecost needs to be added.
The text was updated successfully, but these errors were encountered: