Noticed that when mutation type is a factor, assertions::assert_excludes which tests for zero-length strings fails even though the factor levels contain no empty strings.
This is in part an issue with assertions::assert_excludes, which should work appropriately on factors. We actually have no reason not to autoconvert any factor we get to character, and that might reduce other edge-case issues as well. So we will resolve it in ggoncoplot by autoconverting factor-type mutation type columns to character.
Noticed that when mutation type is a factor, assertions::assert_excludes which tests for zero-length strings fails even though the factor levels contain no empty strings.
This is in part an issue with assertions::assert_excludes, which should work appropriately on factors. We actually have no reason not to autoconvert any factor we get to character, and that might reduce other edge-case issues as well. So we will resolve it in ggoncoplot by autoconverting factor-type mutation type columns to character.
The problematic assertion: