-
Notifications
You must be signed in to change notification settings - Fork 15
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
test failures with TestDyNeuGraph.test_visualize and TestDyNeuGraph.test_visualize_show #4
Comments
Hi @RuixueLiu, thanks for reporting! Can you give me a little more information about your setup? i.e. operating system, python version. Then, I can try to debug this as soon as possible. |
Thanks for the fast response! Here is my setup: 2019-07-11T11:20:41-04:00 CPython 3.6.8 compiler : MSC v.1916 64 bit (AMD64) |
ah, I bet the issue is related to Windows, python's ascii codecs, and calls to I will take a closer look and try to fix this as soon as possible. Thanks again for your help (and patience!) |
Okay, I think I have resolved the issue (see #5 for more about the specific changes). I am going to merge #5 now and hopefully this will quiet the tests. Note, I don't have access to a Windows machine at the moment, so it's possible other issues remain unresolved. @RuixueLiu, you'll have to let me know if you see any other errors (but only if you feel like pulling the latest changes and running the tests again...) |
Thanks Caleb! I pulled the latest change, and the tests can run successfully now! However, I tried to run a simple case in notebook, and found another issue which i believe is due to the Windows file path convention. I can see the index.html in the directory, and can open it from the directory manually. |
Thanks for letting me know. I will look into it asap!
…On Tue, Jul 16, 2019 at 8:52 AM RuixueLiu ***@***.***> wrote:
Another related error:
[image: Capture]
<https://user-images.githubusercontent.com/15878024/61309422-2ce0cc00-a7c0-11e9-86a5-854b5a3eae74.PNG>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#4?email_source=notifications&email_token=ABZJZ4ZQ47JPW75B7HMWWFTP7XVD7A5CNFSM4H7VY6L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2BJYGY#issuecomment-511876123>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABZJZ44GUBRM472UUTDOT4LP7XVD7ANCNFSM4H7VY6LQ>
.
|
Sorry for the delay, @RuixueLiu! I think I resolved what I believe to be the underlying problem, but I haven't been able to test the changes properly. I should have access to a Windows OS soon, and can finish testing the changes then. Thanks again for reporting these issues, and I apologize for the inconvenience. |
hi, i am wondering if there is are any updates on this? |
Sorry for the delay, again. #6 should resolve the issue, but I still don't have access to a Windows machine, so it's possible that the changes do not fully address the issue on Windows and/or have introduced new problems altogether. I went ahead and merged #6 to (hopefully) resolve the issue for Windows users, but again, you'll have to let me know if you are still seeing errors on your end. |
Hi thanks for getting back to me. I pulled the latest changes, but now there are many test failures. |
Hmm... this must be another Windows compatibility issue, because I can't seem to replicate the test failures on my local machine. If you can provide a little more information about (or screenshots of) the failing tests, I'd be happy to help resolve the underlying issues as soon as possible. Sorry (again) for all of the trouble, @RuixueLiu. |
Followed the install instructions using conda, there are two test failures with TestDyNeuGraph.test_visualize and TestDyNeuGraph.test_visualize_show. It seems the errors come with the json.dump function.
![Capture-error](https://user-images.githubusercontent.com/15878024/61002526-b6f7e300-a32f-11e9-83fa-c38e5a0c950a.PNG)
The text was updated successfully, but these errors were encountered: