-
Notifications
You must be signed in to change notification settings - Fork 0
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
Predicates for technical metadata #234
Comments
Additional note: it appears that many of the original properties using the ebucore namespace are now changed, after an ebucore update. So, we can either try to match them up to the new properties or create our own predicates. |
As a precursor, Emily will review which attributes are receiving an example.com predicate vs what is defined here https://github.com/samvera/hyrax/blob/main/config/metadata/hyrax_internal_metadata.yaml |
A couple of files that details the attributes: Those are referenced here: https://github.com/samvera/hyrax/blob/main/app/services/hyrax/characterization/valkyrie_characterization_service.rb |
Brad is going to run the attributes for both FileSet and File metadata to see how values are being assigned. |
@eporter23 Here is the link to my spreadsheet that details what does/doesn't have predicates: https://docs.google.com/spreadsheets/d/1fKTeEAHyzyDuyh_GVzauVQnEXptn4IKAuQtT_jSho3o/edit#gid=0 |
This is great, thanks @bwatson78 ! |
@bwatson78 I've added predicates to everything I think is applicable to us. Note: there are both |
@eporter23 It's funny you mention that. When I was writing the code for our checksum customization, I noticed that |
There's an open issue with Hyrax: samvera/hyrax#6794 |
With #224, I had to create the |
Let me amend my previous statement: it is indexed into a Solr field, but the troublesome piece is that all of the UI interactions only look at |
@eporter23 For |
@eporter23 |
PR made: #316 I have also created a PR with Hyrax to address the fields that match predicates but lack them on the Valkyrie processing side: samvera/hyrax#6820 |
I was trying to check this out in the test environment but I'm running into errors there: #325 - I will try to test in arch however. |
@bwatson78 looking great with the recent changes! I know that fixity checking isn't currently supported in Hyrax, and we'll likely have to do our own local fix for this, and I'm not sure which attribute they previously used. I do see that our hashValue with 3 checksums is populated, though, so I will close this ticket. |
While testing persistence in Fedora, many of our extended technical metadata properties are assigned to an http://example.com namespace. We should adjust this to reflect actual established namespaces where these properties are formally defined.
To discuss: with changes in Valkyrie and FileMetadata, do we need to re-identify RDF predicates for all characterization properties?
Original worksheet with RDF predicates used for Curate:
https://docs.google.com/spreadsheets/d/1csDcDnJdhB1VLlgJUU95DndXSeIP2j8yPMlErrsVVPw/edit#gid=1836172006
Hydra: Works schemas:
https://github.com/samvera/hydra-works/tree/fc6680591ac4bf29309202ca0033c1ed7d778d5c/lib/hydra/works/characterization/schema
Some of our local additions to the Hydra Works schema for Curate:
https://github.com/emory-libraries/dlp-curate/blob/main/app/models/schemas/curate_file_schema.rb
Valkyrie FileMetadata:
https://github.com/samvera/hyrax/blob/main/app/models/hyrax/file_metadata.rb
The text was updated successfully, but these errors were encountered: