Skip to content

Conversation

@kloetzl
Copy link
Contributor

@kloetzl kloetzl commented Jul 18, 2017

Hi,

I am happy to see that there is still work done on MUMmer. Here is a patch, that I had lying around for a while. It greatly increases the performance of postnuc. This pull request also includes the patch to issue #29.

Advertisement Block: You might want to have a look at TUMmer. It is a replacement for the core mummer, that is slightly less accurate, but ten times faster.

Best,
Fabian

kloetzl added 2 commits July 18, 2017 14:13
Previously each `Node` in the alignment graph was wasting huge
amounts of memory. Each `Score` structure had seven bytes of padding
at the end. Thus, roughly a third of the memory occupied by a `Node` is
unused. With the new format, values are tightly packed making a `Node`
smaller. For an unrepresentative sample of two E. coli genomes, the new
postnuc is 10% faster and uses 50% less memory. The former might be due
to better caching.
@peterjc
Copy link
Contributor

peterjc commented Feb 13, 2018

I came across this via https://blog.kloetzl.info/mummer-the/ via Twitter.

As an occasional user of mummer for genome assembly alignment as part of QA, I'd welcome the reported typical 1/3 less memory usage and 20% speed increase.

@shadiakiki1986
Copy link

Ouf this is quite old

@alansill
Copy link

alansill commented Apr 4, 2022

Is there any reason why this hasn’t been merged? Is this code base being maintained any more?

@jdwinkler
Copy link

@kloetzl Thanks for the patch. I am still using Mummer3 and I was wondering if there was an easy way to backport the postnuc changes? I can probably figure it out given enough time and your PR but any tips would be appreciated.

@kloetzl
Copy link
Contributor Author

kloetzl commented Jul 17, 2025

I'm afraid I don't have my mummer3 patch any more. The changes to the structure should be the same, but the changes to the algorithm might be a bit more verbose. Mummer3 is using an older C++ standard so everything is slightly more tedious. Good luck!

@jdwinkler
Copy link

Thanks for replying!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants