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

Provide powerfactor_mode attribute to define inductive and capacitive behaviour #266

Closed
2 of 4 tasks
boltbeard opened this issue Jul 6, 2018 · 1 comment
Closed
2 of 4 tasks
Assignees
Labels
bug Prio1 Priority Level 1

Comments

@boltbeard
Copy link
Contributor

boltbeard commented Jul 6, 2018

The Currently in the components where powerfactor is necessary to push Q to pypsa, there is no way to choose if the powerfactor should exhibit inductive or capacitive behaviour.

This feature becomes essential to reporduce results from various distribution system studies where the conclusion is that more when methods to improve reactive power management is to be

The tasks to ensure this feature is present is:

  • Include a cos_phi_mode attribute to get either inductive or capacitive behaviour (internally use posQ_mode and negQ_mode to ensure inside the code confusion doesn't occur between sign conventions) in the ding0 config files cfg_ding0 under assumptions in cos_phi_load and cos_phi_gen in nodes_to_dict_of_dataframes
  • Modify the calculation methods in nodes_to_dict_of_dataframes
  • Document all the locations where the calculations are changed
  • Document the sign convention for the power factor mode. Make sure to include a self created (to avoid copy right issues) graphic of the PQ plots correctly explaining the sign conventions and what Quadrant each component operats for the different inputs of pf_mode
@boltbeard boltbeard added the bug label Jul 6, 2018
@boltbeard boltbeard self-assigned this Jul 6, 2018
boltbeard pushed a commit that referenced this issue Jul 12, 2018
boltbeard pushed a commit that referenced this issue Jul 19, 2018
…werfactor_mode"

This reverts commit db2478a, reversing
changes made to 84a694f.
@boltbeard boltbeard added the Prio1 Priority Level 1 label Jan 15, 2019
@boltbeard boltbeard added this to the Release 0.1.11-beta milestone Jan 17, 2019
birgits added a commit that referenced this issue Apr 3, 2019
gplssm pushed a commit that referenced this issue Jul 17, 2019
Introduces a power factor mode (inductive/capacitive behaviour) such that all four sectors of the operational circle can be set. Previous defaults remain.

The fix includes commits with the follwing messages:

* Fixes to include weather cell ids and use GeneratorFluctuatingDing0 Objects, export mv_cb states also
* modifications to include metadata as an output which goes into oedb or as a json file when putting out csv
* Added function to grant access to ding0 db tables
* Fixed bug with writing ding0 versioning data
* Modifications to Grant access Database, Todo: Test if it properly works as expected
* Remove 'run_id' from results folder path
* Add weather cell only if at least one generator exists
* Upgrade to DP v0.4.2
* Export aggregated generators individually
* Remove commented lines
* Adapt OEDB export according to changes in
#265
* Let IDs be IDs (and don't mix with other information)
* Replace string in MV/LV_grid_id_db by int
* Revert "Replace string in MV/LV_grid_id_db by int"
This reverts commit 9735f01.
* Revert "Let IDs be IDs (and don't mix with other information)"
This reverts commit 5f4f033.
* Include function to change the sign of Q using powerfactor mode, change config to use this as an assumption
* Add Line Geoms
* Removed geom from LV grids, as there are no geo_data stored in adj_nodes in LV grid edges, Todo: need to use distances to generate them
* Added forgotten imports
* Included the appropriate sign convention that should be chosen in generator and load cases
* Formatting corrections
* Make usage of ids and names consistent
* Adapt table descriptions in docs (only partially)
* Put q_sign in /tools/pypsa_io.py and imported it to /grid/mv_grid/models/models.py, pep 8 changes
* Add docstring for q_sign
* Bring the oedb export back to working order, added LineString Geometries to represent line connections on qgis
* Change name attribute for CSV/OEDB export
Makes consistent names in eDisGo
* Minor code style changes
* Revert "Merge branch 'features/stats-export' into fix/#266-provide_powerfactor_mode"
This reverts commit db2478a, reversing
changes made to 84a694f.
* Resetting results.py to its state in dev
* Shifted q_sign function to obtain load/generator sign convention based sign for the cosine phi to core/powerflow so that it can be imported to other parts of ding0
* Shifted q_sign function to obtain load/generator sign convention based sign for the cosine phi to core/powerflow so that it can be imported to other parts of ding0
* fixed typo
* PEP8
* Added simple test case to check functionality of q_sign
* Reverted changes to voltage_delta_vde formulation, as the sign convention is contained in X
* Changed the sign of the x before voltage_delta_vde call, depending on the function
* Add test file for check_tech_constraints.py
* Changed config to better represent generators that were used in the past to reflect ding0's original use case
* Allowed for changing the sign of the reactive component of voltage based on the given cos_phi
* Wrote a test case for testing voltage_delta_vde
* Changed documentation for references for voltage_delta_vde
* Testing of other parts of the code will be postponned for another time
* Sign convention description in Ding0 to better understand the "inductive" and "capacitive" description of the power factor mode
* Include 'Calculation principles' in docs
* Corrected sign convention in voltage_delta_vde()
* internally, voltage_delta_vde() uses negative sign according to
capacitively operated generators and inductively operated loads
* the reverse case is handled by specification of
cos_phi_gen_mode/cos_phi_load_mode in the config file
* Add sentence about default values and fix typo
* Updated images for cosphi sign convention explanation
* Update What's new and docs
@gplssm
Copy link
Contributor

gplssm commented Jul 17, 2019

Closed by a65b152

@gplssm gplssm closed this as completed Jul 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Prio1 Priority Level 1
Projects
None yet
Development

No branches or pull requests

2 participants