MantisBT - Rosetta
View Issue Details
0000063Rosetta[All Projects] Crashpublic2012-08-08 09:152012-08-08 10:09
smlewis 
smlewis 
normalmajoralways
resolvedfixed 
Trunk 
 
symmetric packing
fixbb.linuxgccrelease @flags_fixbb_symm -s my_monomer -symmetry:symmetry_definition my_symmdef -out:file:silent mysilent.out -out:file:scorefile myscore.sc -linmem_ig 10
50279
0000063: LinearMemoryInteractionGraph causes uninformative crashes in ScoreFunction::operator() when used with symmetry
http://www.rosettacommons.org/content/fix-back-bone-design-fixbb#comment-4933 [^]

Apparently, -linmem_ig breaks symmetric packing. PDInteractionGraph works, LinearMemoryInteractionGraph does not.

I think the bug is that OnTheFlyInteractionGraph (parent of LinearMemoryInteractionGraph) clones an incoming scorefunction incorrectly:

OnTheFlyInteractionGraph.cc:465 score_function_ = new scoring::ScoreFunction( sfxn );

instead of, say,

score_function_ = sfxn.clone();

which would preserve SymmetricScoreFunction's subclassedness.
I am testing if my suggested code change passes the tests, and have requested that the reporting user test against his use case.
No tags attached.
Issue History
2012-08-08 09:15smlewisNew Issue
2012-08-08 09:15smlewisStatusnew => assigned
2012-08-08 09:15smlewisAssigned To => smlewis
2012-08-08 10:08smlewisFixed in SVN Version => 50279
2012-08-08 10:08smlewisNote Added: 0000051
2012-08-08 10:08smlewisStatusassigned => resolved
2012-08-08 10:08smlewisResolutionopen => fixed
2012-08-08 10:09smlewisNote Added: 0000052
2012-08-08 10:09smlewisSummaryLinearMemoryInteractionGraph is not compatible with symmetry => LinearMemoryInteractionGraph causes uninformative crashes in ScoreFunction::operator() when used with symmetry

Notes
(0000051)
smlewis   
2012-08-08 10:08   
This is not exactly fixed - the change to the ScoreFunction type was unintended and incorrect, which has been fixed in 50279. LinearMemoryInteractionGraph is still not compatible with symmetry, but that is explicitly stated in code exposed by r50279, so I'm refiling that as a feature request instead.
(0000052)
smlewis   
2012-08-08 10:09   
(retitled to properly reflect but and discriminate from bug 64)