-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix ragged nested sequence warning #2195
Conversation
Signed-off-by: Foxglove144 <arunabhm@protonmail.com>
Signed-off-by: Foxglove144 <arunabhm@protonmail.com>
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## dev_1.15.0 #2195 +/- ##
==============================================
- Coverage 85.61% 85.30% -0.31%
==============================================
Files 308 308
Lines 27448 27448
Branches 5044 5044
==============================================
- Hits 23499 23415 -84
- Misses 2669 2755 +86
+ Partials 1280 1278 -2
|
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.
Hi @Foxglove144 Thank you very much for fixing this deprecation warning!
Description
Addressed the
VisibleDeprecationWarning
related to creating an ndarray from ragged nested sequences in the script. Modified theanalyze_by_distance
function andanalyze_by_size
function to specifydtype=object
when creating thenp.asarray()
array containingall_assigned_clean
. This ensures that the array can handle objects of different types or shapes without triggering the deprecation warningFixes # (issue)
Type of change
Please check all relevant options.
Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.
Test Configuration:
Checklist