-
Notifications
You must be signed in to change notification settings - Fork 196
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 #161
Comments
@dhirschfeld |
Keep me updated. Having working windows builds is important for the project! |
I've had no joy unfortunately. I've patched When compiling QuantLib I do get hundreds of the below warnings repeated - is that normal?
|
@dhirschfeld which version of VStudio do you use? |
2015 Enterprise |
@dhirschfeld regarding the warnings, I think it should be safe as we don't access directly the attributes that are not exposed to the dll. I will need to take the time to do a build and see if I can reproduce the issue. The latest builds I made where for Python 2 and thus using VS2008. I can't promise anything but will try to do it asap. |
Thanks @dpinte for looking into it. I'd be very interested to know if it's just me or if it doesn't build correctly for py3/win64. I'm happy to help / test / provide any assistance I can but I think I'm a little out of my depth. |
@dhirschfeld can you let me know which py3 version you use? That will help making sure we're testing exactly the same things |
In case it's of interest I also tried a QuantLib build with CMake using the I managed to compile pyql against it but had large numbers of failures. I believe I removed the |
@dhirschfeld thanks a lot for all the information, that will be really useful. What I'll probably do is making sure we have available QuantLib dll's for the standard Python versions that |
That would be fantastic. Let me know if you get it compiled for py3 - I'd be very keen to test it out! |
@dhirschfeld I can reproduce the issue (py3.5/win32). Which boost version do you use? The issue is again the problem of the Settings singleton that does not seem to be shared properly between the pyd's. |
That was 1.61 IIRC |
Are you guys using a patched generate_symbols.py or something? Mine always fails. 1) it thinks the build directory is vc140 (despite msvc9.compiler.VERSION returning: 14.1, it's ignored) when it's actually: QuantLib-1.22/build/v141/ (yes, I hardcoded that instead, still doesn't work; also tried to let it be and copy QuantLib-1.22/build/v141 to QuantLib-1.22/build/vc140/, still fails). Despite abcd.obj being clearly in the directory, the call to: nm_result = subprocess.check_output(command, startupinfo=startupinfo)
I even go further into the process and pass the "command" = nm, --extern-only, --defined-only, abcd.obj and it still fails. How exactly are you generating the symbols on Windows from the obj files? |
@mrslezak IIRC, I was generating the symbols using cygwin, which came with |
I've just built pyql with boost 1.61 and QL master on Win64 and am seeing the below errors:
Are these something to be worried about?
I see that 2 of them have already been reported in #64.
Is
test_bond_schedule_anotherday_bug_cython_implementation
a knownfail - the name is certainly suggestive but there are no other comments which would allow me to deduce what the problem is. It looks like it has been around for a while though 33d9329The text was updated successfully, but these errors were encountered: