-
Notifications
You must be signed in to change notification settings - Fork 27
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
Geospatial - added missing allowDimensionalityMismatch parameter #1531
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1531 +/- ##
==========================================
- Coverage 91.73% 91.73% -0.01%
==========================================
Files 120 120
Lines 15904 15903 -1
==========================================
- Hits 14590 14588 -2
- Misses 1314 1315 +1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work, especially on test coverage! A few comments:
* improved docstrings * passing bool params directly to api because it works * split large tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! Great additions! Left one or two suggestions that I leave up to you ;)
implemented inputs from review: * changed exception handling in tests to use with pytest.raises
@skepticalcat ping 😄 |
Description
Geospatial API comes with parameter "
allowDimensionalityMismatch
" forsearch_features
andstream_features
, which was missing in the python SDK.Added the parameter.
Checklist:
If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.