sequence datatype inputs fix for random.sample calls

DrrDom / crem

CReM: chemically reasonable mutations framework

BSD 3-Clause "New" or "Revised" License

206 stars 38 forks source link

sequence datatype inputs fix for random.sample calls #20

Closed aebrer closed 1 year ago

aebrer commented 1 year ago

should fix #19

DrrDom commented 1 year ago

Thank you for the contribution! It seems that the explicit conversion of a list to a set in row_ids.difference_update(set(selected_row_ids)) is not necessary. If so, could you revert this change back? After that I'll accept a pull request.

aebrer commented 1 year ago

Got it! reverted the change! :)

DrrDom commented 1 year ago

Thank you! I'll accept it right now