Closed MVCowley closed 3 months ago
Issue: For large lists, nn.symdel can take an extended amount of time. It would be helpful to see the progress of the algorithm.
nn.symdel
Solution: Add an optional tqdm progress bar to the main loop of _symdel_lookup.
tqdm
_symdel_lookup
Changes:
progress
symdel
symdel_lookup
enumerate(seq2)
progress=True
tqdm.auto.tqdm
Let me know if you'd like any changes!
No worries! Would you be happy to push a minor release to PyPi so Decombinator can use the new feature?
Issue: For large lists,
nn.symdel
can take an extended amount of time. It would be helpful to see the progress of the algorithm.Solution: Add an optional
tqdm
progress bar to the main loop of_symdel_lookup
.Changes:
tqdm
dependencyprogress
argument to bothsymdel
andsymdel_lookup
enumerate(seq2)
ifprogress=True
, by default it is false and behaviour is unchangedtqdm.auto.tqdm
to support both CLI and notebooksprogress
functionalityLet me know if you'd like any changes!