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
show primitive in embedded document in ListField #3963
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## release/v0.23.3 #3963 +/- ##
===================================================
- Coverage 15.86% 15.86% -0.01%
===================================================
Files 731 731
Lines 81853 81882 +29
Branches 1093 1093
===================================================
+ Hits 12987 12988 +1
- Misses 68866 68894 +28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ 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.
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.
thank you! 🍨
one console log
const pinned3DSample = useRecoilValue(fos.pinned3DSampleSlice); | ||
const activePcdSlices = useRecoilValue(fos.activePcdSlices); | ||
const slices = Boolean(pinned3DSample) && (activePcdSlices?.length || 1) > 1; | ||
|
||
return field && field.ftype !== LIST_FIELD ? ( | ||
const isScalar = useRecoilValue(isScalarValue(path)); | ||
console.log(path, isScalar); |
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.
✂️
c1af0ad
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.
🍨
Nice! Thank you @benjaminpkane and @manivoxel51! 🚀 |
* show primitive in embedded document in ListField * use ListValueEntry for consistency * lint * log --------- Co-authored-by: Benjamin Kane <ben@voxel51.com>
* show primitive in embedded document in ListField * use ListValueEntry for consistency * lint * log --------- Co-authored-by: Benjamin Kane <ben@voxel51.com>
What changes are proposed in this pull request?
Fixes an issue where primitive values in DynamicEmbeddedDocument within ListField is displayed as
None
sample modalHow is this patch tested? If it is not, please explain why.
Verified values are displayed as CSV in the sample modal for the embedded fields:
Release Notes
Is this a user-facing change that should be mentioned in the release notes?
notes for FiftyOne users.
Primitive values in DynamicEmbeddedDocument within ListField will be displayed as comma separated values (previously, displayed as
None
) in the sample modalWhat areas of FiftyOne does this PR affect?
fiftyone
Python library changes