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 functions update_GO_names, cap_names #21

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

Conversation

TylerSagendorf
Copy link

update_GO_names pulls official GO term descriptions from the appropriate OBO file from the Gene Ontology Consortium. Uses data.table for fast replacement of values while merging, but can easily be rewritten with dplyr to remove that import. cap_names capitalizes the first letter of each term description unless there is special capitalization (e.g., "mRNA" would not be capitalized); improves visualizations.

Add data.table, ontologyIndex, and utils to imports. Update Roxygen version.
`update_GO_names` pulls official GO term descriptions from the appropriate OBO file from the Gene Ontology Consortium. Uses `data.table` for fast replacement of values while merging, but can easily be rewritten with `dplyr` to remove that import. `cap_names` capitalizes the first letter of each term description unless there is special capitalization (e.g., "mRNA" would not be capitalized); improves visualizations.
@TylerSagendorf
Copy link
Author

@igordot This addresses issue #19 . I thought this might be helpful, but feel free to scrap this if you don't think it is necessary.

@igordot
Copy link
Owner

igordot commented Aug 9, 2022

Thank you for doing this. This is a really nice enhancement and I am glad you shared the code. I still feel like this is outside the scope of the package and adds some extra dependencies, so I am not ready to merge it, but it's definitely helpful to have it here in case anyone is looking for such functionality.

@TylerSagendorf
Copy link
Author

@igordot No worries. I'll probably make a helper package, then, that has some helper functions for this and the fgsea package. I'll let you know when I've had time to do that.

@igordot
Copy link
Owner

igordot commented Aug 11, 2022

A helper package is a great idea. I thought about suggesting that, but it seemed like an overkill for one function. Makes more sense as you add more functionality.

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