Add addresses as kwarg

[garrison]

Follow-up to #29 (comment).

[garrison]

Caleb, I believe this is what you are looking for. It should be possible to write tests for both the old and new functionality using this, with various combinations of positional arguments and kwargs. One note: There is nothing in this to detect if addresses and bitstring_matrix are provided simultaneously, but a conditional could be added pretty easily to error when this happens.

[caleb-johnson]

Caleb, I believe this is what you are looking for. It should be possible to write tests for both the old and new functionality using this, with various combinations of positional arguments and kwargs. One note: There is nothing in this to detect if addresses and bitstring_matrix are provided simultaneously, but a conditional could be added pretty easily to error when this happens.

Thanks Jim, I think it's fair to prefer the non-deprecated input rather than erroring and document the behavior in the docstring. This doesn't affect anything with backward compatibility since specifying the determinants with positional + keyword arg was not possible until this change.