-
Notifications
You must be signed in to change notification settings - Fork 20
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
Running the docker tests results in an error from SQL Alchemy #11
Comments
Oh hey, good find. I think this is from the most recent merge we did to master </3. Seems to have broken it. Can you check if you get the same before that last merge? Further that should hint on the fix. Thanks! |
There is something definitely going on. I'd been pulling my hair out over it the pass several days as I can't Dejavu to identify by file. I had been playing with the build before Merge #6 and was having success. Along the way I decided to give the fork a try to see how much faster fingerprinting would be. I ran a number of concurrent tests, so in my sloppiness I lost track if I truly ever got the "speed fork" working and when things "broke". Wondering why things all of a sudden stopped working, I spent a few days trying to troubleshoot. With no success I formatted and started afresh on two separate systems, reinstalling following my notes carefully. I was never able to get a working build with the fork or (eventually) the merge. Tonight I downloaded Master 1.2, didn't format the OS (keeping all the modules), re-fingerprinted, and had instant success. |
I can confirm that checking out f113e10 [setup.py fixes] |
I got Merge 5 working with no issues. |
I've done the following:
export PYTHON_VERSION=$(cat Pipfile | awk '/python_version/ {print $3}' | tr -d '"')
docker-compose build
docker-compose run dejavu pipenv run run_tests
The output is a bit too large to include, but it seems there are some errors with the sql query?
If you can point me in the right direction, I'd like to dig into this and fix/PR. Just checking with author first to see what might be obvious?
The text was updated successfully, but these errors were encountered: