search

jeanluct / braidlab

Matlab package for analyzing data using braids
GNU General Public License v3.0
24 stars 10 forks source link

Projectivized coordinates and entropy that uses intaxes #81

Closed jeanluct closed 9 years ago

jeanluct commented 9 years ago

I allowed entropy() to use intaxis/minlength instead of l2 for projectivizing coordinates. The fundamental loop has an extra boundary point that has to be accounted for in intaxis calculation. In non-projectivized coordinates, this is done by subtracting (n-1). In projectivized coordinates, each iterate rescales coordinates, which means that (n-1) has to be properly rescaled too.

At this point, I have marked where the error occurs, and I will get to it, but here's an issue just so I don't forget. :)

In particular, it fails in 

#!matlab
B = braidlab.braid([-4 4 5 -1], 6);
B.entropy(0,1,0,0)
jeanluct commented 9 years ago

From Marko Budisic on 2014-10-27 23:50:05+00:00

braid/entropy: identified and marked error where using intaxis with projectivized coordinates. See Issue #81

→ <>

jeanluct commented 9 years ago

From Marko Budisic on 2014-10-28 20:32:40+00:00

braid/entropy_helper now correctly uses intaxis discounting. Fixes issue #81

→ <<cset 03da0fecf70f>>

jeanluct commented 9 years ago

From Marko Budisic on 2014-10-28 20:32:40+00:00

braid/entropy - MATLAB code accounts for rescaling of length of discounted arc when intaxis is used (See #81)

→ <<cset 8b5eeea63dbd>>