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

[BUG] CSV files unable to upload #139

Open
1 task done
yschan-ntu opened this issue Jul 20, 2023 · 10 comments
Open
1 task done

[BUG] CSV files unable to upload #139

yschan-ntu opened this issue Jul 20, 2023 · 10 comments

Comments

@yschan-ntu
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Description

Hi, am still testing the library! I was trying to upload a testing dataset in CSV. However, I kept getting errors, although am unsure what error I am hitting since there is no error message shown on the portal. I also tried with the datasets provided, but doesn't seem to be a go. Same issue with the models.

Current Behavior

Abortion of upload without a clear error message.

Expected Behavior

I thought that since .CSV is a comma delimited file, it should be able to upload. Minimally, an error message to troubleshoot this issue would be nice.

Steps To Reproduce

I was following the steps on uploading a model dataset. Ideally, there really shouldn't be any issue.

Environment

- Operating System and Version:
- AI Verify Version: 0.9
- Any other relevant information (e.g. version of related package dependencies installed):
As it was built from Docker, thought that this shouldn't be an issue.
Did you build using source code or from docker file?
- From Docker

Screenshots/ Code snippets

Error Message Shown
error_encountered

When I tried to see what the actual error was, so far was unable to get anything.
invalid data files

Additional Context

To note, facing the same issue with models as well since I tried uploading using the default models provided.

Possible Solution (Optional)

No response

@yschan-ntu yschan-ntu added the needs-triage Issues need triages by maintainer label Jul 20, 2023
@kimeetok kimeetok added need-confirmation Issue needs to be reproduced and confirm. and removed needs-triage Issues need triages by maintainer labels Jul 21, 2023
@kimeetok
Copy link
Contributor

Hi @yschan-ntu , we are unable to recreate this bug on our end. Are you able to share the validation error message with us? You can get it by clicking on the row (see screenshot). Meanwhile, perhaps you could try re-building the docker image with no-cache and see if the issue persists. Thank you!
image

@yschan-ntu
Copy link
Author

Clicking on row does not return me any error message as well.

Row_Error

Anyways, I think I will try to rebuild it with no-cache. I will report back on whether this helps.

@yschan-ntu
Copy link
Author

Have already rebuilt with no-cache. Issue seems to have been resolved after rebuilding with no-cache using the 17th June release.

psuedo_data_after_Rebuild

@kimeetok kimeetok added invalid This doesn't seem right and removed need-confirmation Issue needs to be reproduced and confirm. labels Jul 24, 2023
@yschan-ntu
Copy link
Author

A quick update on this bug: seems that this is intermittent. I have faced a similar error on a different day, where datasets and models that are valid were not able to upload without any valid error message. While this resolves itself upon restart of the container (workaround), I would consider this a bug still.

@yschan-ntu yschan-ntu reopened this Jul 27, 2023
@kimeetok kimeetok added need-confirmation Issue needs to be reproduced and confirm. and removed invalid This doesn't seem right labels Jul 27, 2023
@imda-benedictlee
Copy link
Contributor

Hi @yschan-ntu, can I clarify with you what you mean by "datasets and models that are valid were not able to upload without any valid error message"? Also, what are the specific steps that you took to reach the error?

@yschan-ntu
Copy link
Author

yschan-ntu commented Aug 7, 2023

Hi @yschan-ntu, can I clarify with you what you mean by "datasets and models that are valid were not able to upload without any valid error message"? Also, what are the specific steps that you took to reach the error?

As per the screenshot above (https://user-images.githubusercontent.com/133835535/255117197-84f2c22f-3ee2-4d41-97fb-18cfd9f55f60.png), when I uploaded a .CSV file that has previously uploaded without any issues (I tested with a number of file in case the files I tried uploading was corrupted), it gave me an error without any error message. Which is to say, I am unable to troubleshoot any errors with the file even if it has occurred. Note that after restarting, I was able to upload the file successfully.

The steps I have followed are as per the successful datasets standard upload, nothing out of the ordinary.

Note that this is an intermittent issue though.

@imda-benedictlee
Copy link
Contributor

Hi @yschan-ntu, thanks for providing more context on the issue, and apologies for the delay in response. After discussion with the team, we would like your help in providing screenshots of the logs in the Docker containers for the Test Engine App, API GW, and Portal when you encounter the issue again. This will inform us of what actually happens before and during the upload process.

@yschan-ntu
Copy link
Author

Just happened today, please see attached for a copy and paste of the logs at 4.11pm.

2023-08-16 AiVerify-User-Portal-1.txt
2023-08-16 161103 aiverify-user-test-logs.txt

For reference, this is the screenshot of the failed dataset:
Screenshot 2023-08-16 162116

@kimeetok
Copy link
Contributor

Hi @yschan-ntu , thanks for the logs they are helpful in scoping the issue. We are still trying to reproduce the bug on our end, will continue to monitor it and tag it as awaiting more evidence

@kimeetok kimeetok added awaiting-more-evidence and removed need-confirmation Issue needs to be reproduced and confirm. labels Aug 22, 2023
@imda-benedictlee
Copy link
Contributor

Hi @yschan-ntu, I would like to check in to see if this issue still persist for you. If so, would you be able to provide the CSV that you used previously as showed in the screenshot?

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

No branches or pull requests

4 participants