rcsb / symmetry

:ferris_wheel: Detect, analyze, and visualize protein symmetry
GNU Lesser General Public License v2.1
26 stars 16 forks source link

CE-Symm uses trivial alignment in d1n7na1 #33

Closed dmyersturnbull closed 9 years ago

dmyersturnbull commented 10 years ago

http://source.rcsb.org/jfatcatserver/showSymmetry.jsp?name1=d1n7na1

dmyersturnbull commented 10 years ago

Worse cases: d1fsza2, d1fwxb1, d1fwyb1, d1q0qb2, d1gz3b1

lafita commented 9 years ago

This problem might be the same bug that occurred when performing multiple alignments blacking out the previous ones. It was caused because during the optimization step, after the alignment, the black regions were not considered and the aligned fragments could fall inside. This is now fixed in CeSymm.

I tried for all the cases mentioned here and no trivial alignment regions are used. As an example d1n7na1 gives the following output:

d1n7na1_trivial_bug

andreasprlic commented 9 years ago

nice!

dmyersturnbull commented 9 years ago

This fix just made my day! Thanks!