Skip to content

3643 geoconnect guide#3792

Merged
kcondon merged 16 commits intodevelopfrom
3643-geoconnect-guide
May 1, 2017
Merged

3643 geoconnect guide#3792
kcondon merged 16 commits intodevelopfrom
3643-geoconnect-guide

Conversation

@mheppler
Copy link
Copy Markdown
Contributor

New Contributors

Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Thanks!

Related Issues

Pull Request Checklist

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling bc1ee5c on 3643-geoconnect-guide into 64090b7 on develop.

Phrasing and syntax improvements based on Raman's review of the
geoconnect user guide.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling 9b1827a on 3643-geoconnect-guide into 64090b7 on develop.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling a62a4bb on 3643-geoconnect-guide into 64090b7 on develop.

- Explain that we don't recommend sysadmins to install Geoconnect at
  this time.
- For User Guide page, add table of contents (exposed improper nesting,
  also fixed) and explain that Geoconnect is optional.
- Various other tweaks, fixes, and improvements.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling aea9ccf on 3643-geoconnect-guide into 64090b7 on develop.

Copy link
Copy Markdown
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Great work. Much improved from what's in production now. In the User Guide it would be nice to see something about how users can view maps even before we talk about creating them. Maybe some tips on finding maps to look at as well. Please see my other comments in the review itself such as making a dedicated Geoconnect page in the dev guide (like we do with UNF) and other specific comments. I did make a few improvements in aea9ccf

Update: this got buried under "Hide outdated":

Dev Guide: All of this detail here and below about Geoconnect is great but I'm thinking we should have a separate page for it. This page is getting long and the focus should be about setting up a development environment, not a relatively deep dive into a specific feature. I would suggest following the pattern established by UNF, which is even split into multiple pages:

I don't think we want all that detail about UNF on this "Development Environment" page.

Update "mapitlink"
------------------

SQL commands to point a Dataverse installation at different Geoconnect servers:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In the Geoconnect README we say, "You should wait until the WorldMap rewrite is complete before considering installing Geoconnect." Should we explain on this page that people installing Dataverse are welcome to use the two servers below that are hosted by Harvard?

- If you want to set some dataset-specific facets, go to the root dataverse (or any dataverse; the selections can be inherited) and click "General Information" and make choices under "Select Facets". There is a ticket to automate this: https://github.com/IQSS/dataverse/issues/619
Geoconnect works as a middle layer, allowing geospatial data files in Dataverse to be visualized with Harvard WorldMap. To set up a Geoconnect development environment, you can follow the steps outlined in the `local_setup.md <https://github.com/IQSS/geoconnect/blob/master/local_setup.md>`_ guide. You will need Python and a few other prerequisites.

How Dataverse Ingests Shapefiles
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All of this detail here and below about Geoconnect is great but I'm thinking we should have a separate page for it. This page is getting long and the focus should be about setting up a development environment, not a relatively deep dive into a specific feature. I would suggest following the pattern established by UNF, which is even split into multiple pages:

I don't think we want all that detail about UNF on this "Development Environment" page.

@@ -3,29 +3,146 @@
WorldMap: Geospatial Data Exploration
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What percentage of users are going to be viewing maps versus creating them? My guess is that more people will be viewing maps since you don't even need to create a Dataverse account to view a map. You just click the button on the dataset or file page. Given this, before we even talk about creating maps, can we have a short section on viewing maps?

Added a short paragraph explaining how to view a map, per @pdurbin's
suggestion
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling ddff04d on 3643-geoconnect-guide into 64090b7 on develop.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling 7567712 on 3643-geoconnect-guide into 64090b7 on develop.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling fb5028f on 3643-geoconnect-guide into 64090b7 on develop.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 10.103% when pulling abf1775 on 3643-geoconnect-guide into 64090b7 on develop.

@kcondon kcondon merged commit e15eafb into develop May 1, 2017
@kcondon kcondon deleted the 3643-geoconnect-guide branch May 1, 2017 19:24
@pdurbin pdurbin added this to the 4.6.2 - Tabular Mapping milestone May 1, 2017
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.

5 participants