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

GMT parameter transfer files for electrical components #543

Closed
wants to merge 5 commits into from

Conversation

ZhanweiHe
Copy link

GMT parameter transfer files for generator, grid, battery, capacitive load, inductive load, transformer, and converter.

within{{project_name}}.ACACTransformer;
model {{data['model_name']}}
"Isolated transformer template model for GMT level 1 testing"
parameter Modelica.Units.SI.Voltage VHigh = {{data["RMS voltage on high side"]}} "Rms voltage on side 1 of the transformer (primary side)";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have high-side voltage available. See https://docs.google.com/spreadsheets/d/1LHl3iaCbn5cZQt2V28gTz33ga5Y85dZQ/edit#gid=1654231185 and look in the sections for both Transformer and Substation.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm researching to understand if it's possible to provide VHigh some other way. Is it possible to make the model work if I can't find a way to provide that value?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello Nathan,
How about we do not transfer the "VHigh" parameter for "high-side voltage"? The model extends from the existing example model in MBL, which has a default value for this parameter. In the example model, the default value of VHigh is 480 V.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sounds great. Lets do that. Thanks!

Delete the parameter "VHigh" in the transformer .mot file since the high-side voltage is not available in URBANopt side.
@vtnate
Copy link
Contributor

vtnate commented May 4, 2023

@ZhanweiHe I now know where urbanopt sdk stores both high-side and low-side voltages. Since we have default values available I'm happy to use those for now and get the urbanopt output at a later date.

@ZhanweiHe
Copy link
Author

@vtnate Nathan, thank you for letting me about this. To clarify, should we stick with the current version for now and plan to add the parameter "VHigh" at a later date?

@vtnate
Copy link
Contributor

vtnate commented May 31, 2023

This work is being used in #549, so this PR is no longer needed.

@vtnate vtnate closed this May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants