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

add geojson files #111

Merged
merged 1 commit into from
Aug 27, 2019
Merged

add geojson files #111

merged 1 commit into from
Aug 27, 2019

Conversation

bluejuniper
Copy link
Contributor

also includes:

  • conversion script
  • qgis 3 project
  • README

also includes:
* conversion script
* qgis 3 project
* README
@claytonpbarrows
Copy link
Member

Thanks for contributing!
Before merging, can you please explain the overall intent of the PR?
Specifically, I'm hesitant to provide images or additional data to provide references to a specific geography for the test network. I've had several interactions where test system results were erroneously attributed to represent issues faced by power systems in the region. I suspect your contribution has significant value, but I want to be responsible about the encouraging appropriate uses of the dataset.

@bluejuniper
Copy link
Contributor Author

Hi,

I've found the gis representation helpful for visualizing results and the raw dataset. Your concerns are valid, I have deliberately left off a basemap and geopolitical boundary layers for this reason.

-Art Barnes

@bluejuniper
Copy link
Contributor Author

Semi-related, the figure here
https://github.com/bluejuniper/RTS-GMLC/blob/master/node_re_basemap.png
looks like it includes locations for PV plants and possibly other resources. Is that the case, and if yes, are those locations available?

@claytonpbarrows
Copy link
Member

@bluejuniper
I think this should work.
selected_WWSIS-2_sites_withRTSconnections_agg.xlsx
Do you want to amend the PR, or should I merge?

@bluejuniper
Copy link
Contributor Author

I'd suggest to merge now, I can make another PR once I get the PV sited.

@claytonpbarrows claytonpbarrows changed the base branch from master to reformat-DC August 27, 2019 15:30
@claytonpbarrows claytonpbarrows changed the base branch from reformat-DC to master August 27, 2019 15:30
@claytonpbarrows
Copy link
Member

I'll merge. I'm also planning to finish and merge #100 in the coming weeks, so I may request additional changes if needed.

@claytonpbarrows claytonpbarrows merged commit 4173433 into GridMod:master Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants