Python Web Demo Example Broken in Latest Release

[mgh1]

The web demo has the following defects:

1. The parameter mean_file should be mean in the classifier initialization in app.py
2. The mean is treated as a string when it should be an array in pycaffe.py (should add mean = np.load(mean) above the line if mode == 'elementwise':
3. Even after making the code run, the actual result doesn't work, that is, no matter the query image, the result is classified as a "window screen." (I believe the problem lies in the python/caffe code)

[shelhamer]

Right, the web demo needs to be updated to the new Python interface input preprocessing adopted in #816.

The answer to (3) is that the raw_scale needs to be set to 255 as well as the mean fixes you suggested in (1) and (2).

@sergeyk could you fix this if you have a chance?

[marutiagarwal]

This bug is still present in the latest master code. Had to add as suggested:
The mean is treated as a string when it should be an array in pycaffe.py (should add mean = np.load(mean) above the line if mode == 'elementwise'

[shelhamer]

@sergeyk please double-check this and fix master as needed.

[sergeyk]

@shelhamer I think this is in your purview, as you seem to have merged it
to master through a commit: #910

On Fri, Aug 22, 2014 at 10:54 AM, Evan Shelhamer notifications@github.com
wrote:

@sergeyk https://github.com/sergeyk please double-check this and fix
master as needed.

—
Reply to this email directly or view it on GitHub
#899 (comment).

[shelhamer]

All I did was take your fix and release it to master. Taking another look I think the fix is actually correct, and @marutiagarwal just needs to call set_mean() with the array instead of a file path. False alarm.

[sergeyk]

I thinkk you missed one of the commits in the PR though.

On Mon, Aug 25, 2014 at 2:25 PM, Evan Shelhamer notifications@github.com
wrote:

All I did was take your fix and release it to master. Taking another look
I think the fix is actually correct, and @marutiagarwal
https://github.com/marutiagarwal just needs to call set_mean() with the
array instead of a file path. False alarm.

—
Reply to this email directly or view it on GitHub
#899 (comment).

[shelhamer]

No: fc27faf and 39d1e34.

[sergeyk]

ok youre right

On Mon, Aug 25, 2014 at 2:32 PM, Evan Shelhamer notifications@github.com
wrote:

No: fc27faf
fc27faf
and 39d1e34
39d1e34
.

—
Reply to this email directly or view it on GitHub
#899 (comment).