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

Add FAIR Cookbook #541

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add FAIR Cookbook #541

wants to merge 3 commits into from

Conversation

AlasdairGray
Copy link
Member

No description provided.

@AlasdairGray
Copy link
Member Author

@proccaserra are you happy with the listing of the Cookbook on our live deploys page and the associated news article I've drawn up.

- TrainingMaterial
---

The [FAIR Cookbook](https://faircookbook.elixir-europe.org/content/home.html) is an online resource for the Life Sciences with recipes that help you to make and keep data Findable, Accessible, Interoperable and Reusable (FAIR). In keeping with this aim, the Cookbook itself has embedded markup which complies with the Bioschemas [TrainingMaterial profile](https://bioschemas.org/profiles/TrainingMaterial/).

Choose a reason for hiding this comment

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

@AlasdairGray, how is defined compliance to the Profile? I'll assume it means having all attributes where marginality == minimum.
If that's confirmed, we'd need to amend the current json-ld.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ideally all minimal properties should be present. There are some changes that would be needed:

  • Change type from bs:TrainingMaterial to LearningResource
  • Add description
  • Add keywords
  • Make the IRI of the recipe explicit using @id
  • Add a conforms to statement to allow automated validation

Copy link
Contributor

Choose a reason for hiding this comment

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

I looked into the example URL https://w3id.org/faircookbook/FCB010 keywords is present but empty

Copy link
Member Author

Choose a reason for hiding this comment

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

@proccaserra any update on the changes to the FAIR Cookbook?

Copy link
Member Author

Choose a reason for hiding this comment

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

@proccaserra / @terazus we still seem to be missing the keywords element. Other minimal properties have been added.

@ljgarcia
Copy link
Contributor

@AlasdairGray could you please double check this PR? When locally running it, I get FAIRCookbook correctly listed by site
image
However, when I list by profile, I get TrainingMaterial 0.9-DRAFT rather than 1.0-RELEASE
image

@ljgarcia
Copy link
Contributor

ljgarcia commented Dec 9, 2022

@AlasdairGray @proccaserra any news on the issues raised for this PR?

@AlasdairGray
Copy link
Member Author

@AlasdairGray could you please double check this PR? When locally running it, I get FAIRCookbook correctly listed by site image However, when I list by profile, I get TrainingMaterial 0.9-DRAFT rather than 1.0-RELEASE image

In the second screenshot, the FAIRCookbook entry is showing as 1.0-RELEASE. I think you are referring to the header of the by profile section. This will be correct when the branch is synchronised with the master branch (the branch was created before the 1.0-RELEASE).

@AlasdairGray
Copy link
Member Author

@AlasdairGray @proccaserra any news on the issues raised for this PR?

Frustratingly close to being able to list this, see my response above

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

Successfully merging this pull request may close these issues.

3 participants