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

[WIP] Analyse false predictions #168

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

Conversation

Trion129
Copy link
Contributor

@Trion129 Trion129 commented Apr 5, 2018

Fixes #109.

TODO:

  • Load all images and make predictions
  • Compare to extract false predictions
  • Create a GUI to display false predicted images in sequence
  • Create a next and previous button to go back and forth
  • also bind keyboard arrows.
  • Be able to pop the image label if the images are too noisy to learn

@marco-c
Copy link
Owner

marco-c commented May 11, 2018

@Trion129 are you still interested in working on this?

@Trion129
Copy link
Contributor Author

Yep, I just have some quizzes till 18th at college. After that I'll work on this :)

@marco-c
Copy link
Owner

marco-c commented Jul 16, 2018

ping @Trion129

@Trion129
Copy link
Contributor Author

Hi @marco-c, I was busy last month at college. I had made some progress, but I was having issue with placing the "next" button properly, it gets overlapped under the images. I'll push the code anyways, you might have a fix.


img1 = ImageTk.PhotoImage(Image.open("data/" + firefox_name))
panel1.configure(image=img1)
panel1.image = img1
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On adding the image, it covers the next button. Even though they are in separate grids.

@Trion129
Copy link
Contributor Author

Never-mind, it started to work when I put the button inside the image-view itself. Weird tkinter...

@Trion129
Copy link
Contributor Author

@marco-c For the popping of images, as the pics are spread over many csv files, I think it is better to generate a new list "false_pairs.csv" where the image names will be appended. This list can be used by pretrain and train scripts to remove false pairs from data. What's your opinion?

@marco-c
Copy link
Owner

marco-c commented Jul 19, 2018

@marco-c For the popping of images, as the pics are spread over many csv files, I think it is better to generate a new list "false_pairs.csv" where the image names will be appended. This list can be used by pretrain and train scripts to remove false pairs from data. What's your opinion?

I think we shouldn't pop those images, let's just implement the analysis of false predictions.

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.

2 participants