IMP: 'subsample' action renamed to 'subsample_ids'

[hagenjp]

Closes: #299

[lizgehret]

Hey @hagenjp! This mostly looks good, but the tests are failing because you've replaced the name of an external method (Table.subsample) with subsample_ids, which doesn't exist. I'm going to let you find the instances of this and replace them with the original method, because I think it's a good exercise to recognize the difference between external methods that we're calling and our own internal methods (within QIIME 2). In your basecamp message, you already identified where that external method is being called from, so I'll let you extrapolate from there 🙂

Let me know if you have any questions!

[hagenjp]

@lizgehret this pr is ready for round two review from you :)

[lizgehret]

lgtm, just one small comment inline!

[lizgehret]

looks great, thanks @hagenjp!