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

CV title #1147

Closed
floradanna opened this issue Aug 4, 2022 · 4 comments
Closed

CV title #1147

floradanna opened this issue Aug 4, 2022 · 4 comments
Assignees
Labels
cv-ontology define rules for CV and ontology datahub belgiumHub project
Milestone

Comments

@floradanna
Copy link
Collaborator

Describe the bug
When creating a new sample_controlled_vocabs (in both Template and Sample Type), if I give an existing CV title SEEK gives warning error, since CV titles must be unique and no duplicates are allowed.
However, if I look at the list of existing CV titles I see many duplicates (e.g., library source is listed 4 times at least).

To Reproduce
See screenshot below and then:

  1. Go to Create --> Template or Sample type
  2. Click on Attribute Type --> Controlled Vocabulary or Ontology --> use dropdown to see the existing ones
  3. Scroll down and see titles such as "library source" listed multiple times.

This is in contrast with the following error:
Screenshot 2022-08-04 at 14 11 11

Expected behavior
Consistent behaviour, although I don't know yet what that would be. To be discussed.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]

Additional context
In DataHub, the same attribute (like, library source) with the same CV list (titled library source as well) will be "used" many many times by derived templates (via Create Template based on existing templates) and derived sample types (via applying the same template to many studies or assays). So, I think this is what it is causing having repeated CV list titles in the dropdown menu.
However, this is in contrast with default SEEK rule that prohibits having repetition of the same title in CV list.

Questions:

  • does SEEK only check if the CV title exist, independently of the listed terms being identical or different? probably, yes.
  • should the content of the list be checked?
  • is this a big limit for users since all CV lists are listed together, meaning CV lists created by anyone in any project at any point in time in a given SEEK instance?
  • should lists be also restricted somehow (by project, by template...)?
  • should lists make use of unique identifiers (seek ID) to check unicity?
  • how does DataHub differentiate between Ontology and CV? Currently, the GUI to create an Ontology list is identical to the one for creating a CV list.
  • It might be necessary now to define what's an Ontology list vs a CV list
    • Ontology list:
      • only one root ontology term and all children are fetched
      • no editing allow (no deletion, no addition of free terms)
      • Ontology list take the name of the root term as title
    • CV list:
      • no direct fetching of ontology terms from OLS (but still possible to paste URL to a ontology term)
      • possibility to add, delete, edit terms
      • free text for CV list title
@floradanna floradanna added the datahub belgiumHub project label Aug 4, 2022
@floradanna
Copy link
Collaborator Author

@rabuono something to figure out

@floradanna
Copy link
Collaborator Author

  • Add sharing permission to CV list: by default the same as the Sample type that uses it (sharing permission for Sample type are still to be implemented).

  • Keep possibility

    • to add terms to the CV list via Edit ISA study and Edit ISA Assay (even if samples exist)
    • to delete terms from the CV list if the terms are not already used
  • hide dropdown for CV

@kdp-cloud kdp-cloud moved this to Todo in DataHub Mar 20, 2023
@floradanna floradanna removed the status in DataHub Jul 14, 2023
@floradanna floradanna moved this to Requirements in DataHub Jul 26, 2023
@floradanna floradanna moved this from Requirements to Backlog in DataHub Nov 9, 2023
@floradanna floradanna moved this from Backlog to Todo in DataHub Mar 11, 2024
@kdp-cloud
Copy link
Collaborator

@kdp-cloud kdp-cloud moved this from Todo to Backlog in DataHub Jul 2, 2024
@NielsGeudens NielsGeudens moved this from Backlog to Todo in DataHub Jul 2, 2024
@kdp-cloud kdp-cloud added this to the 1.16.1 milestone Dec 4, 2024
@kdp-cloud
Copy link
Collaborator

kdp-cloud commented Jan 7, 2025

The schema has a template_id field which is not being used in any of the DataHub instances. Instead, the relation to the Templates has to go through the TemplateAttributes, similar to the SampleTypes.

A new issue for this has been created: #2103

@kdp-cloud kdp-cloud moved this from Todo to In Progress in DataHub Jan 10, 2025
@kdp-cloud kdp-cloud moved this from In Progress to Coded in DataHub Jan 10, 2025
@kdp-cloud kdp-cloud self-assigned this Jan 10, 2025
@github-project-automation github-project-automation bot moved this from Coded to Merged in DataHub Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cv-ontology define rules for CV and ontology datahub belgiumHub project
Projects
Status: Merged
Development

No branches or pull requests

2 participants