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

fix bugs in ipynb2py and app test #385

Closed
wants to merge 38 commits into from

Conversation

edieYoung
Copy link
Contributor

switch the order of test apps to verify the error.

@edieYoung
Copy link
Contributor Author

edieYoung commented Jun 11, 2018

jenkins not passed:
run app tests 2 failed
http://10.239.47.211:18888/view/ZOO-PR/job/ZOO-PR-Python-AppTests/65/console

@edieYoung
Copy link
Contributor Author

jenkins not passed:
run app tests 4 successfully, test 2 failed:
http://10.239.47.211:18888/view/ZOO-PR/job/ZOO-PR-Python-AppTests/66/console

@edieYoung
Copy link
Contributor Author

jenkins passed:
test 2 has passed, because the download path was wrong, it worked well after revising to $FTP_URI/analytics-zoo-models-new/object-detection/analytics-zoo_ssd-mobilenet-300x300_PASCAL_0.1.0.model.
test 1 still not passed.
console output:
http://172.168.2.101:8080/view/ZOO-PR/job/ZOO-PR-Python-AppTests/68/console

@edieYoung edieYoung changed the title switch test order to verify fix bug in ipynb2py and app test Jun 13, 2018
@edieYoung edieYoung changed the title fix bug in ipynb2py and app test fix bugs in ipynb2py and app test Jun 13, 2018
@edieYoung
Copy link
Contributor Author

edieYoung commented Jun 13, 2018

jenkins not passed:
http://172.168.2.101:8080/view/ZOO-PR/job/ZOO-PR-Python-ExampleTests/367/console
textclassification, objectdetection and customized loss and layer have passed.
the predict.py of imageclassification has bug:
File "/opt/work/jenkins/workspace/ZOO-PR-Python-ExampleTests/pyzoo/zoo/examples/imageclassification/predict.py", line 38, in predict
print("\t%s, %f" % (labelMap[sortedProbs[999 - i][1]], sortedProbs[999 - i][0]))
IndexError: list index out of range

@edieYoung
Copy link
Contributor Author

As for apps test: Revised the download path's filename "variational_encoder" to "variational-encoder", as well as the related calls in the notebook scripts.

@edieYoung
Copy link
Contributor Author

jenkins passed: app test has finised
http://172.168.2.101:8080/view/ZOO-PR/job/ZOO-PR-Python-AppTests/84/console
time used:
#1 anomaly-detection-nyc-taxi time used:192 seconds
#2 object-detection time used:853 seconds
#3 ncf-explicit-feedback time used:190 seconds
#4 wide_n_deep time used:315 seconds
#5 using_variational_autoencoder_and_deep_feature_loss_to_generate_faces time used:699 seconds
#6 using_variational_autoencoder_to_generate_faces time used:435 seconds
#7 using_variational_autoencoder_to_generate_digital_numbers time used:252 seconds
#8 sentimentAnalysis time used:1487 seconds
#9 image-augmentation time used:131 seconds

@edieYoung edieYoung closed this Jun 21, 2018
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.

1 participant