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

jsonld-context-autogen missing the base IRI #88

Open
tekrajchhetri opened this issue Nov 20, 2024 · 3 comments
Open

jsonld-context-autogen missing the base IRI #88

tekrajchhetri opened this issue Nov 20, 2024 · 3 comments
Assignees

Comments

@tekrajchhetri
Copy link
Contributor

tekrajchhetri commented Nov 20, 2024

@puja-trivedi @djarecka I noticed the context file is missing the default IRI.

Issue:
Whenever you do the conversion of JSON-LD to ttl file, the file path is taken as base IRI, while it should be something like http://example.org/NIMP/<ID>..
Screenshot 2024-11-20 at 1 02 35 PM

In the previous version, it's not the case.
Screenshot 2024-11-20 at 1 03 22 PM

@tekrajchhetri
Copy link
Contributor Author

tekrajchhetri commented Nov 20, 2024

Update:

The issue is in the JSON-LD file. So, there is no need to update the context file.

In current, i.e., rapid release file: We do not have any prefix and also no base IRI causing the aforementioned issue. I suggest to make use of prefix, explicitly rather than having base, as is the best practice.

      "was_derived_from": [
        "EC-VYUUTE137687",
        "EC-UYBXXW324656"
      ]

In old JSON-LD file: We have NIMP prefix so everything's fine.

"was_derived_from": [
        "NIMP:LA-GFHFDE415771LOMHPL202182",
        "NIMP:LA-LYZZRW951279LOMHPL202182",
        "NIMP:LA-QVRSQW524460LOMHPL202182",
        "NIMP:LA-BFDXZY775863LOMHPL202182",
        "NIMP:LA-VUVPPX234898LOMHPL202182",
        "NIMP:LA-HHEABG177343LOMHPL202182", 
      ],

@puja-trivedi
Copy link
Collaborator

puja-trivedi commented Nov 22, 2024

@tekrajchhetri - I took a look at the library_generation model and the NIMP prefix is already set to http://example.org/NIMP/. See line#18 -> https://github.com/brain-bican/models/blob/main/linkml-schema/library_generation.yaml.

While we did not explicitly define the prefix in the spreadsheet; linkml automatically fills in any missing prefix (that is why it is not missing from the .yaml file).

@tekrajchhetri
Copy link
Contributor Author

@puja-trivedi Yes, there exist NIMP prefix but in the JSON-LD files that you've shared with me, they're missing the NIMP prefix as I've pointed out in the earlier example (#88 (comment)). The same is shown in the image below, which is a snapshot of a JSON-LD file. Also, you can see from the image blow that not in all cases NIMP prefix is missing.

image

It could be the issue with the translator that you're using to generate JSON-LD files. Also a suggestion, I think it would be better to add some automated checks to ensure there's always a prefix associated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants