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

[Current Task] Submit a CSV file #20

Open
1 task done
MickeyZeng opened this issue Jul 2, 2020 · 6 comments
Open
1 task done

[Current Task] Submit a CSV file #20

MickeyZeng opened this issue Jul 2, 2020 · 6 comments
Assignees
Labels
Important This issue has to be solved improvement function improvement new feature

Comments

@MickeyZeng
Copy link
Owner

MickeyZeng commented Jul 2, 2020

Add a new function to submit the csv file which must be a legal format, otherwise, the system cannot process the file in right way. After submit the file, the system can have image preview, and user can click the submit to process the current image and check if there is any differences between the result of prediction and index in CSV file or not. If there are no difference, the prediction table would have special sign for the correct prediction.

The file should contain:

  1. the URL(local address) of the image.
  2. the index(the label) of image.

Update:
  • Small bug needs to be fixed, see down below or click for more detail
@MickeyZeng MickeyZeng changed the title [New Feature] Submit a csv file [New Feature] Submit a CSV file Jul 2, 2020
@MickeyZeng MickeyZeng added the improvement function improvement label Jul 2, 2020
@MickeyZeng MickeyZeng added the Important This issue has to be solved label Jul 8, 2020
@MickeyZeng
Copy link
Owner Author

The front-end send a request include the address and ground truth of image, and back-end would find this image in local computer. After all, the image will be sent to front-end to display.

@jakecastelli jakecastelli changed the title [New Feature] Submit a CSV file [Current Task] Submit a CSV file Jul 8, 2020
@MickeyZeng
Copy link
Owner Author

MickeyZeng commented Jul 14, 2020

There has a special situation. When the ground truth is not in the top 5 results proved by neural network, the system should have display area to let the user know what the ground truth is. @jakecastelli @MickeyZeng


This issue also includes #7

@jakecastelli
Copy link
Collaborator

jakecastelli commented Jul 14, 2020

TODO:

  • If the gt result is not in top 5, we have the 6th column for the gt

  • Previous Image / Next Image Button Functionalities

jakecastelli added a commit that referenced this issue Jul 17, 2020
Now, if user chooses to upload CSV file and provide the ground
truth, it will be shown in the leader board. #20

TODO:
If the ground truth is not in the leader board, show the ground truth in
the 6th row
@jakecastelli
Copy link
Collaborator

jakecastelli commented Jul 18, 2020

For Upload_CSV, I think you did not give me the ground truth label only for the first image / the first time @MickeyZeng .
Because When I was testing CSV, I found the first image always did not have label, from the second image onward then will be normal.

  • Fixing The Bug For CSV, The first image does not have gt provided

Update:
The problem was front-end js, the first time received image blob, did not save the gt, problem solved.

jakecastelli added a commit that referenced this issue Jul 18, 2020
@MickeyZeng
Copy link
Owner Author

There are some bugs in this process.

  1. Submit pic given by SCV would have an error about the index.
  2. Because the label in SCV is number instead of label name, so the errors are reported here.

jakecastelli added a commit that referenced this issue Aug 12, 2020
Added a check for custom network.
Resolved the issue in #20.
Link: #20 (comment)
@jakecastelli
Copy link
Collaborator

Resolved this one not issue #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Important This issue has to be solved improvement function improvement new feature
Projects
None yet
Development

No branches or pull requests

2 participants