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.

Sign up for GitHub

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

Jump to bottom

Changes to make the fast CTRE index builder work for OSM (do not merge) #404

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Changes to make the fast CTRE index builder work for OSM (do not merge) #404

wants to merge 2 commits into from
+30 −6

Commits on Jul 8, 2021

  1. Changes to make CTRE index builder work for OSM 

    1. Assume that all triples are subject-predicate-object (no , or ;
syntac to avoid repeating the subject and/or predicate) and that all
triples are on a single line and end with a newline.
2. Make sure that literals of type ^^geo:wktLiteral can get externalized
(they were recognized as xsd values so far, which is a bug)
3. Externalize all literals of length > 128 (so far the threshold was
1024). This will externalize all larger geometries, but not POINTs and
simple geometries like buildings.
4. Don't run the TurtelParserTest because it has several tests
concerning the escaping of , and ; which now fail due to 1.
    @hannahbast
    hannahbast committed Jul 8, 2021
    Configuration menu
    Copy the full SHA
    c3892ca View commit details
    Browse the repository at this point in the history

  2. Make clang-format happy

    @hannahbast
    hannahbast committed Jul 8, 2021
    Configuration menu
    Copy the full SHA
    58f66e3 View commit details
    Browse the repository at this point in the history