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

20171213 Docs interlis updated #50

Merged
merged 18 commits into from
Mar 30, 2019
Merged

20171213 Docs interlis updated #50

merged 18 commits into from
Mar 30, 2019

Conversation

sjib
Copy link
Contributor

@sjib sjib commented Dec 13, 2017

Various minor changes based on open issues:
#44 (closed now)
#33
Change Link to zip with Demodata #49 (closed now)


Multiedit
---------
If you want to edit attributes for multiple objects of the same class you can use the `mulitedit Tool of QGIS <https://docs.qgis.org/2.18/en/docs/user_manual/working_with_vector/attribute_table.html#multi-edit-fields>`_.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

mulitedit -> multiedit

@sjib
Copy link
Contributor Author

sjib commented May 24, 2018

Can this be merged?

- *SIA405_Base.ili*
- *VSA_DSS_2015_2_d_20170602.ili* for Version 2015 –> corrected version relations

>>> java -jar ili2pg.jar --createEnumTxtCol --schemaimport --importTid --sqlEnableNull --createEnumTabs --createFk --noSmartMapping --dbdatabase qgep --dbschema vsa_dss_2015_2_d_391 --dbusr postgres --dbpwd sjib --log createschema_VSA_DSS_2015_2_d_391.log VSA_DSS_2015_2_d_20170602.ili
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be formatted as code (.. code:: I think)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No - I think this looks fine - see preview


Specify the path variable to postgresSQL on your system

``-u`` sets the user
Copy link
Contributor

Choose a reason for hiding this comment

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

Capital or not (difference between example above and explanation here)


* qgep schema of demodata has missing foreignkeys: Einleitstelle.fs_Gewaessersektor, Versickerungsanlage.fs_Grundwasserleiter, Abwasserknoten.fs_hydrgeomref –> check whether ok in empty schema ``qgep``.

* Need to use ``ST_Force3D`` for geometries – because qgep model is now with 3D coordinates
Copy link
Contributor

Choose a reason for hiding this comment

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

remove the word "now"

@m-kuhn
Copy link
Contributor

m-kuhn commented May 25, 2018

Just some minor comments from quickly looking through the proposed changes

Copy link
Contributor Author

@sjib sjib left a comment

Choose a reason for hiding this comment

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

en/installation-guide/database-initialization.rst can be skipped - is corrected in the meantime.

@sjib sjib merged commit 4cc2eb5 into master Mar 30, 2019
@3nids 3nids deleted the docs_interlis branch September 6, 2019 10:57
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.

2 participants