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

Pp/#753 new test grid for simona #757

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from

Conversation

pierrepetersmeier
Copy link
Contributor

@pierrepetersmeier pierrepetersmeier commented Feb 28, 2024

Resolves: #753

New Branch publishing, because false naming pattern.
Thanks for the tip @danielfeismann

@pierrepetersmeier pierrepetersmeier self-assigned this Feb 28, 2024
@pierrepetersmeier pierrepetersmeier added the enhancement New feature or request label Feb 28, 2024
@pierrepetersmeier
Copy link
Contributor Author

pierrepetersmeier commented Feb 28, 2024

Does the error (#644 "line length is 50% greater...") occur in other networks as well? @danielfeismann

@pierrepetersmeier
Copy link
Contributor Author

The line where the warning occurs has the same coordinates as in #644, i.e. the "distance" 0.

@pierrepetersmeier
Copy link
Contributor Author

Okay, if we change the coordinates minimally and they are no longer exactly the same, the warning disappears.

@danielfeismann
Copy link
Member

Okay, if we change the coordinates minimally and they are no longer exactly the same, the warning disappears.

Would be fine for me

@pierrepetersmeier
Copy link
Contributor Author

Okay, if we change the coordinates minimally and they are no longer exactly the same, the warning disappears.

Would be fine for me

should I do the same in vn_Simona? Maybe it's not quite as clean because there's more than one line, but then the warnings are gone.

@danielfeismann
Copy link
Member

danielfeismann commented Feb 29, 2024

Since I guess it wouldn't fit too often it will not help that much at vn_simona. But if you want give it a try, feel free.

@pierrepetersmeier
Copy link
Contributor Author

Since I guess it wouldn't fit too often it will not help that much at vn_simona. But if you want give it a try, feel free.

Yes, that didn't work :/

@danielfeismann
Copy link
Member

This looks very good already @pierrepetersmeier. Thanks for the steps done here, it made it very easy for me to make the following adjustments:

  • moved the grid to the TU Dortmund side
  • moved both WECs to the upper voltage level to reduce the voltage rise in the lv grid 😄
  • changes in config (changes from main branch and write output files)

More things could be done (e.g. include heat pumps). But I think we should stick with this for now.
Will bring this into our internal discussions if it suits our needs and will get back to you with the result.

@pierrepetersmeier
Copy link
Contributor Author

This looks very good already @pierrepetersmeier. Thanks for the steps done here, it made it very easy for me to make the following adjustments:

  • moved the grid to the TU Dortmund side
  • moved both WECs to the upper voltage level to reduce the voltage rise in the lv grid 😄
  • changes in config (changes from main branch and write output files)

More things could be done (e.g. include heat pumps). But I think we should stick with this for now.
Will bring this into our internal discussions if it suits our needs and will get back to you with the result.

alright thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Test Grid for Simona
3 participants