search

KevinSee / PITcleanr

Compress PIT tag capture histories for use in various analyses
https://kevinsee.github.io/PITcleanr/
Creative Commons Attribution 4.0 International
8 stars 6 forks source link

Option in buildConfig() to use _D, _M, _U suffixes instead of default _A0 and _B0 #29

Closed mackerman44 closed 11 months ago

mackerman44 commented 11 months ago

Bear with me... remining myself of how to updated functions, roxygenize documentation, and create a pull request to merge branches. A little rusty.

Better way's to skin this cat, but hoping to add an option to buildConfig() to use _D, _M, and _U suffixes for downstream, middle, and upstream arrays, respectively, rather than the default _A0 and _B0. Feel free to improve upon as you see fit.

Related, I would be interested in an option or argument to secondarily merge the _M arrays to the _D or _U arrays, if desired.

mackerman44 commented 11 months ago

Kevin - Please decline this pull request. I'm instead going to start a new "development" branch on my fork and update these functions in a better manner. I don't love how I went about these changes after some thought.