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.

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

prefix_map from config should be combined with prefixes in Excel #205

Closed
dalito opened this issue Feb 21, 2024 · 1 comment
Closed

prefix_map from config should be combined with prefixes in Excel #205

dalito opened this issue Feb 21, 2024 · 1 comment
Labels
enhancement Request for changing or improving existing features
Milestone

Comments

@dalito
Copy link
Member

dalito commented Feb 21, 2024

...and added to the Excel prefix table if missing.

Currently the prefix map has to be added to Excel by hand. Only the entires from the prefix map in Excel will be used. Even if you define "myp" as prefix in config, a CURIE in Excel myp:0001 is not understood until "myp" is defined again in the prefix sheet in Excel.

@dalito
Copy link
Member Author

dalito commented Feb 3, 2025

Solved by #257 but slightly different:

The default prefixes from rdflib are combined with the ones from idrange.config. The prefixes sheet is not read anymore but just used to show the prefixes known and usable.

@dalito dalito closed this as completed Feb 3, 2025
@dalito dalito added this to the 0.9.0 milestone Feb 3, 2025
@dalito dalito added the enhancement Request for changing or improving existing features label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Request for changing or improving existing features
Projects
None yet
Development

No branches or pull requests

1 participant