search

maxfrei750 / paddle

Toolbox for the easy, deep learning-based primary particle size analysis of agglomerated, aggregated, partially sintered or simply occluded particles.
MIT License
10 stars 1 forks source link

mpac.zip is not a zip file #5

Closed Rajarshi1001 closed 2 years ago

Rajarshi1001 commented 2 years ago

I have tried running the notebook __analyze_dataset.ipynb__ . While executing the cell image I get the following issue, the dataset seems to be private or remove by the owner. image

maxfrei750 commented 2 years ago

Apparently, the university broke my download link. Thanks for letting me know. I just committed a fix. Please pull the new main version and let me know if it works now.

Rajarshi1001 commented 2 years ago

I tried running the train function but it initially asked for the wandb API key. This gave the following error image

Could you please give a detailed instructions to run the notebooks because there seems to multiple issues in the repo

Rajarshi1001 commented 2 years ago

the file is not a zip file issue still persists

Rajarshi1001 commented 2 years ago

the load model also throws an error image

maxfrei750 commented 2 years ago

There were indeed a few errors, which resulted from changes in the code base. That's what I get for not having tests and not using docker for this project. I fixed all problems and on my system, all demos run flawlessly again. That being said, I was not able to reproduce the File is not a zip file. So please pull the recent commits, recreate your conda environment (see updated instructions in readme), test again and let me know the results.

maxfrei750 commented 2 years ago

@Rajarshi1001 Did the update of the demos fix your File is not a zip file error? If so, then please close this issue.

maxfrei750 commented 2 years ago

Closing, since the problem seems to be resolved.