Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prateek Murgai: Small changes for the system. #27

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

prateek64
Copy link

No description provided.

prateek64 added 9 commits November 13, 2014 22:10
Changed 'runObjectProposals' to 'runObjectProposal'
New fields have been added which might be of use of further evaluation.
Possible function to fix the bug in PlotMetric.m : No getMethods() found
in PlotMetric
Function that loads paths for pascal images, best candidate,
imagenet_2013 etc .
Precalculated files which can be used for plotting purposes, while
comparing with ground truth. Use these files when running PlotMetric.m
Proposals calculated for endres for the demo images
Chnaged name in the M file from 'greedy_matching_rowwise' to 'maxRow'
No getMethods() found in the repository which is being called in the
plotMetric function. A possible bugfix is mentioned as comments in the M
file.
This field  is being called in the compute_best_recall_candidates ()
function which is not present in your current methods.mat file.
@nchavali
Copy link
Contributor

Prateek, can you please take a look at the new README file and make changes according to it.
We don't use methods.mat and plotMetric.m at all. I have reoved those files. sorry for the confusion

@dexter1691
Copy link
Member

Also, @prateek64 : if you could create another branch for each feature you patch/fix. Merging into the main branch is not a good idea, we can always merge the work into main branch after we are sure its good to be added for a stable release.

@prateek64
Copy link
Author

I will go through the new ReadMe also , @nchavali , Please can you change the 'runObjectProposals()' to 'runObjectProposal()' in your ReadMe in ii) and iii) under the Generating Proposals heading. Your function name is without the 's'. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants