Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

DNase not DNAse #169

Closed
turbomam opened this issue Feb 10, 2022 · 6 comments
Closed

DNase not DNAse #169

turbomam opened this issue Feb 10, 2022 · 6 comments
Assignees

Comments

@turbomam
Copy link
Member

turbomam commented Feb 10, 2022

@mslarae13 I'd like to make that capitalization change throughout SNTC

see https://en.wikipedia.org/wiki/Deoxyribonuclease

@turbomam turbomam self-assigned this Feb 10, 2022
@turbomam turbomam changed the title DNase not DNAase DNase not DNAse Feb 10, 2022
@mslarae13 mslarae13 moved this to 🔖 Ready in SubPort Squad Issues Jan 27, 2023
@mslarae13
Copy link

@turbomam what is SNTC?
JGI's template has DNAse so I think we need to leave it like JGI has it.

Where did you see it should be DNase?

@mslarae13
Copy link

DNase is conventionally correct. Update NMDC schema to be correct. @sujaypatil96 's work on mapping the terms will them map DNase to DNAse for the JGI export.

@turbomam
Copy link
Member Author

SNTC = @turbomam's abbreviation for @mslarae13's Soil NMDC Terms Compiled google sheet

by the way

@turbomam
Copy link
Member Author

Implement this in sheets_and_friends and/or jgi_mg.yaml etc.

@mslarae13
Copy link

  • jgi_mt.yaml

Mark will make the update.. some of the titles have DNase written correctly, but it's wrong in some of the comments. check and update for consistent use as DNase

@turbomam
Copy link
Member Author

Schema files updated. Will check sheets_and_friends now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: ✅ SubPort 1 - Done
Development

No branches or pull requests

2 participants