dthuerck / mapmap_cpu

A high-performance general-purpose MRF MAP solver, heavily exploiting SIMD instructions.
BSD 3-Clause "New" or "Revised" License
102 stars 51 forks source link

bug in prev_level() #21

Closed cdcseacave closed 3 years ago

cdcseacave commented 4 years ago

there is clearly a bug in prev_level() though I am not sure of the right fix (depends if m_previous refers to positive or negative previous); I guess the fix should be m_previous = (m_level ? &m_levels[m_level - 1] : NULL), but pls confirm:

https://github.com/dthuerck/mapmap_cpu/blob/master/mapmap/source/multilevel.impl.h#L152

dthuerck commented 4 years ago

Hi there,

thanks for reporting this and sorry for my delay, I'm pretty busy ATM. I'm guessing this occurs when using mvs-texturing? If so, is there anyway I can get some (small) input data to replay the problem on my side?

Thanks!

dthuerck commented 3 years ago

push

Could you provide me with your input data in order to replicate the problem?

cdcseacave commented 3 years ago

sry, I gave up trying to make this work, and I removed all the source code related to it