A couple of FactorGraph unit tests are buggy

[iglesias]

Valgrind detects quite a few errors for conditional jumps or moves depending on unitialised values in FactorGraph.evaluate_enery_param_data and FactorGraph.evaluate_enery_param_data_sparse, see output of
valgrind --leak-check=full ./shogun-unit-test --gtest_filter=FactorGraph.evaluate_energy_param_data_*

These tests are also leaking memory.

@hushell, let's try to fix this :)

[iglesias]

I have temporarily disabled these tests in #1311.

[hushell]

@iglesias I'll fix ASAP.

[vigsterkr]

can we close this one?

[hushell]

Sure. I'll take a look later today.

On Tue, Feb 4, 2014 at 4:45 AM, Viktor Gal notifications@github.com wrote:

can we close this one?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1298#issuecomment-34055683
.

/**

• ============HuShell============
  */

[hushell]

@iglesias I tried valgrind --leak-check=full ./shogun-unit-test --gtest_filter=FactorGraph.evaluate_energy_param_data_*

There is no problem on my laptop. Shall we close this?

[iglesias]

Sure, it looks fine!