Minor combineTCR fix & refactor combineBCR

[Qile0317]

Switched back to using the $barcode syntax for selecting the barcode column instead of assuming its in the first position. Encountered this issue when working with a custom version of the data I manipulated.

[ncborcherding]

only change is that the sample argument was disallowed to be null and the stop call was removed when it is

You will need to make sure this doesn't cause errors when combining multiple BCR experiments. The reason sample was required was to avoid duplicate barcodes when using igraph.

[Qile0317]

only change is that the sample argument was disallowed to be null and the stop call was removed when it is

You will need to make sure this doesn't cause errors when combining multiple BCR experiments. The reason sample was required was to avoid duplicate barcodes when using igraph.

Ok - I may be completely mistaken but im a little confused on how immediately injecting a stop call when samples = NULL isn't equivalent for any use case but I'll revert.

[Qile0317]

@ncborcherding re-added functionality for samples = NULL.

[ncborcherding]

Sorry I think I was confused with the NULL, i see what you did now.