-
Notifications
You must be signed in to change notification settings - Fork 33
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
lineage only for some operators? #92
Comments
We saw this as a lineage vs. versioning balance. In theory, lineage only cares about major geoprocessing steps. Something like clean is better stored in the versioning of a city model, in my opinion. It was also a first step in automatic metadata generation and started with specific major processes with the hope that it might inspire others. Metadata can be subjective so users can also make their own decisions in the end about which steps they wish to include. I think the discussion about lineage vs. versioning is an interesting one and one that is on going. I'm not against other steps being included though, and I hope that by not including them I'm not suggesting that they shouldn't be included, just that we didn't prioritise them at the time for lineage. The more metadata the better. |
One thing to add here, from a developer's perspective, is that adding a lineage item falls under the responsibility of the individual commands and might require different implementation per operation. So, we should keep track of that. Unfortunately, the boundaries of what should be in lineage and how it's documented aren't always clear, afaik myself, so it's something to think about while moving forwards and try and improve as we implement more things in cjio. |
New behaviour of cjio is the following (d094f85): If One can simply add the And one can delete the I also bluntly decided to start the name of cjio with the property and
Lineage for |
Why certain commands are included in the lineage (eg merge, filter_lod), while others are not (eg. clean), when all of them modifies the citymodel.
The text was updated successfully, but these errors were encountered: