Skip to content

Conversation

@m-mohr
Copy link
Contributor

@m-mohr m-mohr commented Apr 11, 2021

Would be great if someone could check the list of potential formats and add/improve others.

@emmanuelmathot
Copy link
Member

It looks good to me. Simple but I like the idea to keep the expression open to any object. I have the feeling that this will open to many formats in the future and could require proper extensions for the expression spec.

@emmanuelmathot
Copy link
Member

How do we specify that the field can also be used in the virtual:assets items' fields?

@m-mohr
Copy link
Contributor Author

m-mohr commented Apr 20, 2021

Not sure what we decided for, but I think we could either go with the existing field in raster that just re-uses this definition or use processing:expression in the raster extension. The idea was to have a common building block, so both sound good to me.

@m-mohr
Copy link
Contributor Author

m-mohr commented May 5, 2021

From my side we can merge this... Thoughts?

@emmanuelmathot
Copy link
Member

We can merge

@m-mohr m-mohr merged commit b0dc14b into main May 18, 2021
@m-mohr m-mohr deleted the expression-object branch May 18, 2021 15:16
@m-mohr
Copy link
Contributor Author

m-mohr commented May 18, 2021

Merged.

@m-mohr m-mohr added this to the 1.1.0 milestone Jul 23, 2021
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.

3 participants