-
Notifications
You must be signed in to change notification settings - Fork 303
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
Support 'unknown' sdtype #1532
Support 'unknown' sdtype #1532
Conversation
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.
Should we also change the else
case on line 149 that defaults to categorical
now that we're adding the unknown
sdtype?
Hi @frances-h, in which file is it? In this PR we change the metadata detection to detect |
b668d95
to
cc6fa97
Compare
Codecov ReportPatch coverage:
❗ 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. Additional details and impacted files@@ Coverage Diff @@
## metadata-improvment #1532 +/- ##
=======================================================
+ Coverage 96.17% 96.49% +0.32%
=======================================================
Files 48 48
Lines 4107 4028 -79
=======================================================
- Hits 3950 3887 -63
+ Misses 157 141 -16
☔ View full report in Codecov by Sentry. |
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!
b7a4791
to
90957fc
Compare
* def support unknown * add test * not pii 2
* def support unknown * add test * not pii 2
* def support unknown * add test * not pii 2
* def support unknown * add test * not pii 2
* def support unknown * add test * not pii 2
Resolve #1516