mgijsberti / synoptic

Automatically exported from code.google.com/p/synoptic
0 stars 0 forks source link

Enable IntrBy by default only for Perfume #360

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
With other Synoptic projects apart from Perfume, the IntrBy invariant is 
untested both in correct functioning and usefulness. Therefore, it should only 
be enabled by default in Perfume.

Fixing this will involve creating a commandline flag (default false) in 
synoptic.main.options.SynopticOptions that allows the user to enable/disable 
IntrBy. Somehow, it should default to true if enablePerfDebugging==true, 
although it won't work to set this inside SynopticOptions.

Original issue reported on code.google.com by tonyohm...@gmail.com on 28 Jan 2014 at 9:26

GoogleCodeExporter commented 9 years ago
Check how much is needed, as Issue362 should render this (almost) useless?

Original comment by sebastia...@gmail.com on 6 Feb 2014 at 6:50

GoogleCodeExporter commented 9 years ago
Oops, you're right. This issue is defunct, as it is subsumed by Issue362, as 
you mentioned.

Original comment by tonyohm...@gmail.com on 6 Feb 2014 at 7:11

GoogleCodeExporter commented 9 years ago

Original comment by tonyohm...@gmail.com on 6 Feb 2014 at 7:12