Skip to content

Add w3curie type and edit def of curie to be less restrictive #221

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ialarmedalien
Copy link

@ialarmedalien ialarmedalien commented Mar 27, 2025

Adding in the new w3curie type.

I tried to update the repo to use py3.12 with the current versions of linkml, linkml-runtime, and pyyaml so that I could regenerate the project files. Run make gen-project gave the following output:

# for all the files in the schema folder, run the gen-python command and output the result to the top
# level of the project.  In other repos, we'd include mergeimports=True, but we don't do that with
# linkml-model.
ValueError:  Unknown argument: slot_usage = {'dimensions': {'equals_number': 1}}
ValueError:  type "any_number" must declare a type base or parent (typeof)
cp staging/*.py linkml_model
poetry run gen-project -d staging --config-file gen_project_config.yaml linkml_model/model/schema/meta.yaml
Traceback (most recent call last):
[...snip a load of callback lines...]
ValueError: The ifabsent value `default_range` of the `range` slot could not be processed
make: *** [gen-project] Error 1

Looks like the various project generators, etc., have become more restrictive since linkml-model was last updated.

ETA: just seen the series of PRs to update the model repo. I can rebase this PR and generate the new files once the deps and other bits and pieces are fixed and merged.

@ialarmedalien ialarmedalien marked this pull request as ready for review March 27, 2025 19:13
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.

1 participant