Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Fix geodata table input #970

Merged
merged 37 commits into from
Aug 23, 2021
Merged

Fix geodata table input #970

merged 37 commits into from
Aug 23, 2021

Conversation

dchassin
Copy link
Member

@dchassin dchassin commented Aug 7, 2021

This PR fixes issues with csv-table2glm-object converter when using geodata. The correction of these problems is demonstrated with a new geodata tutorial included in this PR.

Current issues

There is a known problem with the gldcore/geodata/powerlines.py package. See arras-energy/gridlabd#49 for details.

Code changes

  • Add support for module option, fix class handling in gldcore/converters/csv-table2glm-object.py, and other minor functional improvements.
  • Fix gldcore/converters/csv2glm.py to support -M|--module <MODULE> support.
  • Fix map zoom bound check problem in gldcore/converters/json2html.py, add support for hidden objects, fix occasional obj error, and add support for object properties map_color, map_weight, and map_opacity.
  • Add tutorial folder to gldcore/geodata with 6 examples of using table to object converter. The notebook gldcore/geodata/docs/Geodata.ipynb is also updated.
  • Fix gldcore/scripts/gridlabd-convert error handling input file types.
  • Fix typo, name field handling, and nan usage in gldcore/scripts/gridlabd-geodata.

Documentation changes

Test and Validation Notes

  • Update gldcore/converters/autotest/test_table*.glm to reflect corrected table usage and fix data folder path.
  • Add gldcore/geodata/tutorial folders with examples of usage.
  • Remove spurious space in header of gldcore/converters/autotest/table2glm_input.csv

@dchassin dchassin added the bug Request a fix or change to the code label Aug 7, 2021
@dchassin dchassin marked this pull request as ready for review August 9, 2021 19:58
@dchassin dchassin mentioned this pull request Aug 23, 2021
17 tasks
@aivanova5 aivanova5 self-requested a review August 23, 2021 18:16
@aivanova5 aivanova5 merged commit dface88 into develop Aug 23, 2021
@aivanova5 aivanova5 deleted the develop-fix-geodata-table-input branch August 23, 2021 18:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Request a fix or change to the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants