-
Notifications
You must be signed in to change notification settings - Fork 58
Add photovoltaic(GFL) and correct photovoltaic(GFM) #52
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
base: master
Are you sure you want to change the base?
Conversation
yy-Hu-xjtu
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.
除上述反馈意见外,其余文件经审核测试无误。
注:处理反馈意见重新提交pull request前请注意把README.md里面的版本时间进行更新。
(Other than the feedback, the remaining files have been checked and are correct.
Note: Before re-submitting the pull request to address the feedback, please ensure the version date in README.md is updated.)
| Para.v_dc_ref | ||
| Para.fvdc; | ||
| Para.fidc]; | ||
| if Type ==40 |
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.
1.需要增加本类型机组的简介注释
(The explanatory comments about the apparatuses you added is necessary.)
| Para0041.fpll =15; % (Hz) pll bandwidth | ||
| Para0041.fidq =600; % (Hz) current control bandwidth | ||
| Para0041.C_dc =1; | ||
| Para0041.w0 =W0; |
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.
1.建议在 "=" 和 "W0"间加入一个空格。
(Missing blank between "=" and "W0")
UserMain.m
Outdated
| % UserDataName = 'BessInfiniteBus'; % Single battery energy storage system and infinite bus | ||
| % UserDataName = 'PVInfiniteBus'; % Single Photovoltaic and infinite bus | ||
| % UserDataName = 'PVGfmInfiniteBus'; % Single Photovoltaic(GFM) and infinite bus | ||
| % UserDataName = 'PVGflInfiniteBus'; % Single Photovoltaic(GFL) and infinite bus |
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.
1.建议在“PV”和“Gfm”之间补充一个“”,这样命名或许更容易理解一些。
(Adding a "" between "PV" and "Gfm" is recommended to make it easier to comprehense.And so do "PV" and "Gfl".)
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.
1.由于新增了一个独立的跟网型光伏机组,本构网光伏机组的命名是否应考虑改为如“PhotovoltaicGFM”为宜?
(Due to the addition of a new independent grid-following photovoltaic apparatus, should the naming of this grid-forming photovoltaic apparatusbe considered for a change, such as "PhotovoltaicGFM"?)
Add photovoltaic(GFL) and correct photovoltaic(GFM)
Add photovoltaic units that adopt grid-following control, and correct the power setting issues of grid-forming photovoltaic units.