R-CMD Check : selectLongestDuplicate, plotEstimatedWallTimes, removeTails, condenseRepeatedDomains, plotStackedLineage

[Klangina]

• pick_longer_duplicate selectLongestDuplicate: no visible binding for global variable ‘AccNum’
• pick_longer_duplicate selectLongestDuplicate: no visible binding for global variable ‘row.orig’
• plot_estimated_walltimes plotEstimatedWallTimes: get_powerset : : no visible global function definition for ‘combn’
• plot_estimated_walltimes plotEstimatedWallTimes: no visible binding for global variable ‘n_inputs’
• plot_estimated_walltimes plotEstimatedWallTimes: no visible binding for global variable ‘est_walltime’
• plot_estimated_walltimes plotEstimatedWallTimes: no visible binding for global variable ‘advanced_opts’
• remove_tails removeTails: no visible binding for global variable ‘domains_keep’
• repeat2s condenseRepeatedDomains : no visible binding for global variable ‘.’
• stacked_lin_plot plotStackedLineage: no visible binding for global variable ‘cpcols’
• stacked_lin_plot plotStackedLineage : no visible global function definition for ‘unit’

Originally posted by @the-mayer in #4 (comment)

[Klangina]

Fixed all the warnings except:

removeTails: no visible binding for global variable ‘domains_keep’
@jananiravi @the-mayer : Need the logic for extracting domains_keep from prot, kindly guide.

[the-mayer]

As the keep_domains param defaults to FALSE, it seems like this bit of code rarely (if ever) gets evaluated. Regardless, this is a good find! I'm going to convert your comment to another issue, while merging the other changes in your PR.