-
Notifications
You must be signed in to change notification settings - Fork 17
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
Can't determine whether an API call is from TensorFlow when more than two functions are in a call chain #49
Comments
@tatianacv Can you open a PR with the failing test? |
@tatianacv Can you please add a PR like this one to https://github.com/wala/ML/pulls? |
Sorry, I know I asked for this, but I think we can use ponder-lab#26 for now. |
The passing test case for this issue is here: ML/com.ibm.wala.cast.python.ml.test/source/com/ibm/wala/cast/python/ml/test/TestTensorflowModel.java Line 97 in 7e18125
Input file: ML/com.ibm.wala.cast.python.test/data/tf2n.py Lines 1 to 14 in 7e18125
|
* Replace commented out output with "fine" logging. * Use "fine" logging in Maven build. * Split logging properties for CI. Less logging on the CI. CI is complaining about too much logging. * Centralize log config. So that it applies to all tests. * Fix CI. * Define a custom logging config file property.
* Replace commented out output with "fine" logging. * Use "fine" logging in Maven build. * Split logging properties for CI. Less logging on the CI. CI is complaining about too much logging. * Centralize log config. So that it applies to all tests. * Fix CI. * Define a custom logging config file property. Co-authored-by: Manu Sridharan <msridhar@gmail.com>
When processing the following:
We found an error in determining whether we are dealing with a TensorFlow API. We found that the tensor analysis is empty:
ML/com.ibm.wala.cast.python.ml.test/source/com/ibm/wala/cast/python/ml/test/TestTensorflowModel.java
Line 82 in 93c96d8
The text was updated successfully, but these errors were encountered: