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

Deprecated operation_3206 added topic Whole genome methylation analysis #802

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LucieLamothe
Copy link
Collaborator

FIX #770

Co-authored-by: vbaillet vbaillet@noreply.users.github.com

/!\ Change replace by of deprecated operation_3206 when updated new topic ID

FIX edamontology#770

Co-authored-by: vbaillet vbaillet@noreply.users.github.com
@sonarcloud
Copy link

sonarcloud bot commented Dec 23, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@LucieLamothe
Copy link
Collaborator Author

The handling of deprecation of term and change of sections (from operation to topic) in biotools and other edam user need to be discussed before making this change

@matuskalas
Copy link
Member

There are 29 records using operation_3206 on Bio.tools, so we need to figure out what to do with them

  • Does the automated update/obsoleting/replacement in Bio.tools need to be made more sophisticated?
  • Do we (anyway) need to review those entries "manually"?

@sonarcloud
Copy link

sonarcloud bot commented Jun 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@matuskalas
Copy link
Member

matuskalas commented Jun 21, 2022

After talking with @hansioan and giving it some more thought, I'm growingly convinced that we should use consider for replacement concepts that are from another section of EDAM (like operation to topic in this case).
@LucieLamothe it'll also be awesome to include that in a corresponding validation test in the CI.

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.

[Edam Browser User] Change proposition for http://edamontology.org/operation_3206
2 participants