CSAILVision / CSAILVision/places365

About confusions in run_placesCNN_unified.py

Open
#87 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
2.1k
Forks
538
PR merge metrics
No merged PRs in 30d

Description

In this py file, when get `weight_softmax`, there is a statement `weight_softmax[weight_softmax<0] = 0` and I think maybe this is to remove feature maps with negative effects. However, as `weight_softmax[weight_softmax<0] = 0` is implemented, parameters in the network also changes. Is it a good trend?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in run_placesCNN_unified.py at the weight_softmax assignment and the subsequent negative-value masking statement. Trace whether this operation shares storage with network parameters, then check the surrounding model and visualization flow. Done means establishing whether parameters are modified and documenting or correcting the behavior accordingly.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
computer-vision, machine-learning
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.