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

Snf review roy2020 #37

Merged
merged 5 commits into from
Apr 17, 2024
Merged

Snf review roy2020 #37

merged 5 commits into from
Apr 17, 2024

Conversation

StevenCannon-USDA
Copy link
Collaborator

I changed all yml files, adding a field "scientific_name" to each (since some files describe genes from several species), and added a two-letter prefix to each gene_symbol - for example, Gm, Mt, Lj. Also removed two very incomplete files.

StevenCannon-USDA added 2 commits April 14, 2024 18:35
…lete or well supported (incomplete work from Greg)
…er genus-species prefix (e.g., Mt, Gm, Lj) to gene symbols
@StevenCannon-USDA
Copy link
Collaborator Author

@ScottKalberer - I am converting this PR to draft status, since I see more changes that I would like to make.

@StevenCannon-USDA StevenCannon-USDA marked this pull request as draft April 15, 2024 13:29
StevenCannon-USDA added 2 commits April 15, 2024 09:39
…pecies, I separated these by species, prefixing each and moving it into its respective species directory. Also updated the templates.
@StevenCannon-USDA StevenCannon-USDA marked this pull request as ready for review April 15, 2024 14:44
@StevenCannon-USDA
Copy link
Collaborator Author

Scott - this is ready for your review, please. However, I recommend NOT looking at each change, since I changed ALL files (79 I believe)! We can discuss the types of changes and make further corrections incrementally as needed.

Copy link
Collaborator

@ScottKalberer ScottKalberer left a comment

Choose a reason for hiding this comment

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

I agree with your deletion of these two gene function files.

Copy link
Collaborator

@ScottKalberer ScottKalberer left a comment

Choose a reason for hiding this comment

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

I agree with this addition of the field "scientific name" and the prefixing of the Genus-species to the gene symbols. I also see that the field classical locus is being eliminated.

Copy link
Collaborator

@ScottKalberer ScottKalberer left a comment

Choose a reason for hiding this comment

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

I accept this addition of the gensp prefix to the titles of YML files and the splitting of multi-species files into single-species files.

@ScottKalberer ScottKalberer merged commit 90d5cb1 into main Apr 17, 2024
2 checks passed
@ScottKalberer ScottKalberer deleted the snf_review_Roy2020 branch April 17, 2024 21:23
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.

2 participants