Adding "<" operator to unitigIterator class and "[]" operator to CompactedDBG class

pmelsted / bifrost

Bifrost: Highly parallel construction and indexing of colored and compacted de Bruijn graphs

BSD 2-Clause "Simplified" License

201 stars 25 forks source link

Adding "<" operator to unitigIterator class and "[]" operator to CompactedDBG class #38

Open samhorsfield96 opened 3 years ago

samhorsfield96 commented 3 years ago

Would it be possible to add a "<" operator to the unitigIterator class and a "[]" operator to CompactedDBG class?

I am implementing a function with openMP multithreading where I am iterating over unitigs using a for loop. However, openMP dictates that the for loop be in the format for (unitigIterator i = cdbg.begin(); i < cdbg.end(); i++) which is not currently possible to implement.

Many thanks in advance, Sam.

GuillaumeHolley commented 3 years ago

Hi @samhorsfield96,

Sorry about the delay, for some reasons I missed the notification of this issue until today. Implementing those operators is feasible I think but would require some important refactoring. Unfortunately I am the only person working on Bifrost at the moment so I'll add this to my todo list but I wouldn't expect it to be released any time soon.

Guillaume