8000 Change metadata handling to be compatible with OpenVBI by selimnairb · Pull Request #66 · CCOMJHC/WIBL · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Change metadata handling to be compatible with OpenVBI #66

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

Merged
merged 5 commits into from
May 5, 2025

Conversation

selimnairb
Copy link
Member

This PR changes metadata handling in wibl-python when WIBL files are processed into GeoJSON files, as described in #65. This represents initial work to achieve compatibility with OpenVBI by implementing similar metadata handling directly in wibl-python code (rather than making OpenVBI a dependency of wibl-python and using OpenVBI's metadata handling code directly). This PR also includes an update to documentation in README.md describing the metadata format change, and provides a simple script to convert to the new OpenVBI-compatible format.

In future work, the OpenVBI metadata handling code can be used directly by wibl-python. However, doing so will ideally require that OpenVBI first be packaged via conda-forge (since wibl-python is also packaged as conda-forge package). However, until this conda packaging work has been done for OpenVBI, the current PR will ensure compatibility.

…t OpenVBI-compatible metadata; Add migration script for converting to OpenVBI compatible metadata
@selimnairb selimnairb linked an issue Mar 26, 2025 that may be closed by this pull request
@selimnairb
Copy link
Member Author

@HaleyAddison Please give this PR as well as issue #65 a look when you have a chance as this will require changes the metadata files you use in your workflows. Let me know if you have any questions. Thanks!

…N conversion since these are not required elements and we are now deep merging user metadata with the template, so non-required elements should not be in the template
Copy link
Collaborator
@brian-r-calder brian-r-calder left a comment

Choose a reason for hiding this comment

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

This looks good; the compatibility is worth it, I think.

@selimnairb selimnairb merged commit b0dcd76 into main May 5, 2025
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.

Update wibl-python to use OpenVBI-compatible B-12 metadata
2 participants
0