search

FREEDM-DGI / FREEDM

The distributed grid intelligence manages power transactions and attached physical devices.
http://www.freedm.ncsu.edu/
17 stars 18 forks source link

Please correct the semantics regarding m_grossPowerFlow #369

Closed mcatanzaro closed 10 years ago

mcatanzaro commented 10 years ago

Hey Li,

From what I've gathered from our discussions, LBAgent::m_grossPowerFlow represents some quantity that is related to, but distinct from, the gross power flow. This is super confusing. Can you please either rename the variable to reflect what it actually is (the summation of the square of each flow?), or else modify the usage of the variable such that it only stores the actual gross power flow?

My concern is that if we have a variable named m_grossPowerFlow, it really needs to store the gross power flow and nothing else.

mcatanzaro commented 10 years ago

Li, has this been resolved?

scj7t4 commented 10 years ago

No longer called grossPowerFlow in #384