-
Notifications
You must be signed in to change notification settings - Fork 18
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
Vector symbology api #163
Vector symbology api #163
Conversation
5dd3a54
to
daf25ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat!!
I just have a single comment/question, otherwise I think it's good to merge
# TODO Load source-layer properly, from qgis symbology? | ||
try: | ||
source_layer = get_source_layer_names(url)[0] | ||
layer_parameters["sourceLayer"] = source_layer | ||
except ValueError: | ||
pass | ||
# TODO Load style properly | ||
|
||
layer_parameters.update(type="fill") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we keep this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure the source_layer
stuff isn't being used anywhere right now, so it should be fine to get rid of. The type
is required in the schema so we should leave that for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok! Thanks for your answer
Let's merge this then!
Changes
geojson
notebook API to use color expressions. Also loads colors for vector tile layers fromqgz
files, and exports color information when exporting toqgz
.