Skip to content
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

Minor fixes #3283

Merged
merged 1 commit into from Jul 11, 2023
Merged

Minor fixes #3283

merged 1 commit into from Jul 11, 2023

Conversation

NeoKish
Copy link
Contributor

@NeoKish NeoKish commented Jul 11, 2023

  • No. You can skip the rest of this section.
  • Yes. Give a description of this change to be included in the release
    notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)

What areas of FiftyOne does this PR affect?

  • App: FiftyOne application changes
  • Build: Build and test infrastructure changes
  • Core: Core fiftyone Python library changes
  • Documentation: FiftyOne documentation changes
  • Other

Copy link
Contributor

@benjaminpkane benjaminpkane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! @NeoKish 🙇 🚀

@benjaminpkane benjaminpkane changed the base branch from develop to release/v0.21.3 July 11, 2023 19:18
@benjaminpkane benjaminpkane changed the base branch from release/v0.21.3 to develop July 11, 2023 19:18
@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Patch coverage: 62.03% and project coverage change: +0.24 🎉

Comparison is base (ad43b14) 15.41% compared to head (b4a0598) 15.65%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3283      +/-   ##
===========================================
+ Coverage    15.41%   15.65%   +0.24%     
===========================================
  Files          564      564              
  Lines        69658    69656       -2     
  Branches       667      681      +14     
===========================================
+ Hits         10739    10908     +169     
+ Misses       58919    58748     -171     
Flag Coverage Δ
app 15.65% <62.03%> (+0.24%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ackages/core/src/components/Actions/ActionsRow.tsx 0.00% <0.00%> (ø)
...ckages/core/src/components/Schema/SchemaSearch.tsx 0.00% <0.00%> (ø)
...ages/core/src/components/Schema/SchemaSettings.tsx 0.00% <0.00%> (ø)
...ore/src/components/Sidebar/Entries/FilterEntry.tsx 0.00% <0.00%> (ø)
app/packages/state/src/hooks/index.ts 100.00% <ø> (ø)
app/packages/state/src/recoil/aggregations.ts 36.36% <0.00%> (ø)
app/packages/state/src/hooks/useSchemaSettings.ts 14.05% <3.73%> (-0.85%) ⬇️
.../packages/state/src/recoil/schemaSettings.atoms.ts 49.19% <49.19%> (+49.19%) ⬆️
...ackages/state/src/hooks/useSchemaSettings.utils.ts 100.00% <100.00%> (ø)
app/packages/state/src/recoil/index.ts 100.00% <100.00%> (ø)
... and 2 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@benjaminpkane benjaminpkane merged commit 2f5e53e into voxel51:develop Jul 11, 2023
9 of 11 checks passed
benjaminpkane pushed a commit that referenced this pull request Jul 11, 2023
benjaminpkane added a commit that referenced this pull request Jul 12, 2023
* Refactor schemaControls, use selectField() when searching, cleaner schema documentation

selection mode is default now for filed visibility

* update count dependency to refelct when applying filter rules

* add documentation links

* better disable field handling - (un)select is broken - fixing next

* improve select all control

* uncheck subpaths even if they are disabled

* show embedded document in selection mode

* field doc type using tertiary text color

* minor tweaks in color modal title style

* review comments and fix include nested fields selection view

* review comments, refactor selection row, and count fix

* add first vitest for visibility, upgrade vitest packages

* add basic test and refactor more

* keep one embed doc field

* no coverage when yarn test

* add support for multi-select with string type

* adds a Docker Hub Pulls badge

* Typo

* tweaking

* lint

* Add files via upload

* Update brain.rst

* Update index.rst

* Update index.rst

* Update index.rst

* Update index.rst

* Release v0.21.2 (#3251) (#3255)

* update sidebar test, fix small regressions (#3250)

* remove next index ref, fix string filter is matching default (#3249)

* release notes

* package bumps

* disabled e2e

* lint

* fix mac arm64

* Teams v1.3.2 release note

* rm e2e

* add main

* add icon

* typo

* first pass at milvus integration docs

* adding milvus to user_guide/brain

* adding to integrations index

* adding milvus logo

* square image, lower resolution

* editing pass

* consistency

* bugfixes

* disable field unit testing

* group and video dataset unit tests disabled

* toPatches disabling tests

* toClips

* label types test

* more granular field selection when disabling label subfields

* geolocation, heatmap, segmentation tests

* complete disabling test

* review comments

* enable vector fields, disable top level id field, more grnular disabling of FrameSupportField and FrameNumberField

* disable list of valid labels from visibility toggle

* review comments

* linting

* editing pass

* tweaks

* cleanup

* fixing capitalization

* adding new vector integrations

* bumping package versions

* adding release notes

* or count label tags (#3267)

* has group slices only when group dataset (#3262)

* Updating `Session.close()` (#3253)

* kill server on session close

* close desktop app

* cleanup

* lint

* lint

* is_open

* rm remote warning

* Embedded frame label fixes (#3256)

* add frame cases to dynamic label tests

* embedded frame label fixes

* linting

* adding a merge_sample() method

* Minor typo fixes

* adding test for one() method

* updating release notes

* tweak

* documenting list bucket perms

* Fix sidebar matching on label fields (#3270)

* only matches

* db_field bug and path fix

* Suppress errors due to none fields (#3275)

* don't throw error

* add debug msg

* import order

* Fixing #3277 (#3279)

* fixing filter_keypoints() bug

* adding dynamic doc test

* interactive typo fix for Dave

* removing persistent dataset usage in unit tests

* Sidebar filtering tests and fixes (#3280)

* only matches

* db_field bug and path fix

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* tweaks

* exclude no only matches

* add to release notes

* whitespace

* updating release notes

* Minor fixes (#3283)

* docs tweaks

* documenting cache=True feature

---------

Co-authored-by: manivoxel51 <mani@voxel51@gmail.com>
Co-authored-by: Lanny W <lanzhenwang9@gmail.com>
Co-authored-by: manivoxel51 <109545780+manivoxel51@users.noreply.github.com>
Co-authored-by: Brian Moore <brian@voxel51.com>
Co-authored-by: imanjra <ibrahim@voxel51.com>
Co-authored-by: topher <topher@voxel51.com>
Co-authored-by: Jacob Marks <jamarks13@gmail.com>
Co-authored-by: brimoor <brimoor@umich.edu>
Co-authored-by: Ayush Chaurasia <ayush.chaurarsia@gmail.com>
Co-authored-by: Kishan Savant <ksavant577@gmail.com>
Co-authored-by: Kacey <kacey@voxel51.com>
Co-authored-by: Kishan Savant <66986430+NeoKish@users.noreply.github.com>
benjaminpkane added a commit that referenced this pull request Jul 12, 2023
* Release v0.21.2 (#3251)

* update sidebar test, fix small regressions (#3250)

* remove next index ref, fix string filter is matching default (#3249)

* release notes

* package bumps

* disabled e2e

* lint

* fix mac arm64

* Teams v1.3.2 release note

* rm e2e

* add main

* adding new vector integrations

* bumping package versions

* adding release notes

* or count label tags (#3267)

* has group slices only when group dataset (#3262)

* Updating `Session.close()` (#3253)

* kill server on session close

* close desktop app

* cleanup

* lint

* lint

* is_open

* rm remote warning

* Embedded frame label fixes (#3256)

* add frame cases to dynamic label tests

* embedded frame label fixes

* linting

* adding a merge_sample() method

* Minor typo fixes

* adding test for one() method

* updating release notes

* tweak

* documenting list bucket perms

* Fix sidebar matching on label fields (#3270)

* only matches

* db_field bug and path fix

* Suppress errors due to none fields (#3275)

* don't throw error

* add debug msg

* import order

* Fixing #3277 (#3279)

* fixing filter_keypoints() bug

* adding dynamic doc test

* interactive typo fix for Dave

* removing persistent dataset usage in unit tests

* Sidebar filtering tests and fixes (#3280)

* only matches

* db_field bug and path fix

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* tweaks

* exclude no only matches

* add to release notes

* whitespace

* updating release notes

* Minor fixes (#3283)

* docs tweaks

* documenting cache=True feature

* Release/v0.21.3 (#3282)

* Refactor schemaControls, use selectField() when searching, cleaner schema documentation

selection mode is default now for filed visibility

* update count dependency to refelct when applying filter rules

* add documentation links

* better disable field handling - (un)select is broken - fixing next

* improve select all control

* uncheck subpaths even if they are disabled

* show embedded document in selection mode

* field doc type using tertiary text color

* minor tweaks in color modal title style

* review comments and fix include nested fields selection view

* review comments, refactor selection row, and count fix

* add first vitest for visibility, upgrade vitest packages

* add basic test and refactor more

* keep one embed doc field

* no coverage when yarn test

* add support for multi-select with string type

* adds a Docker Hub Pulls badge

* Typo

* tweaking

* lint

* Add files via upload

* Update brain.rst

* Update index.rst

* Update index.rst

* Update index.rst

* Update index.rst

* Release v0.21.2 (#3251) (#3255)

* update sidebar test, fix small regressions (#3250)

* remove next index ref, fix string filter is matching default (#3249)

* release notes

* package bumps

* disabled e2e

* lint

* fix mac arm64

* Teams v1.3.2 release note

* rm e2e

* add main

* add icon

* typo

* first pass at milvus integration docs

* adding milvus to user_guide/brain

* adding to integrations index

* adding milvus logo

* square image, lower resolution

* editing pass

* consistency

* bugfixes

* disable field unit testing

* group and video dataset unit tests disabled

* toPatches disabling tests

* toClips

* label types test

* more granular field selection when disabling label subfields

* geolocation, heatmap, segmentation tests

* complete disabling test

* review comments

* enable vector fields, disable top level id field, more grnular disabling of FrameSupportField and FrameNumberField

* disable list of valid labels from visibility toggle

* review comments

* linting

* editing pass

* tweaks

* cleanup

* fixing capitalization

* adding new vector integrations

* bumping package versions

* adding release notes

* or count label tags (#3267)

* has group slices only when group dataset (#3262)

* Updating `Session.close()` (#3253)

* kill server on session close

* close desktop app

* cleanup

* lint

* lint

* is_open

* rm remote warning

* Embedded frame label fixes (#3256)

* add frame cases to dynamic label tests

* embedded frame label fixes

* linting

* adding a merge_sample() method

* Minor typo fixes

* adding test for one() method

* updating release notes

* tweak

* documenting list bucket perms

* Fix sidebar matching on label fields (#3270)

* only matches

* db_field bug and path fix

* Suppress errors due to none fields (#3275)

* don't throw error

* add debug msg

* import order

* Fixing #3277 (#3279)

* fixing filter_keypoints() bug

* adding dynamic doc test

* interactive typo fix for Dave

* removing persistent dataset usage in unit tests

* Sidebar filtering tests and fixes (#3280)

* only matches

* db_field bug and path fix

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* base image sample tests

* cleanup

* exclude bug

* rm onlyMatch

* frame and dynamic tests

* adding coverage

* keypoints fixes

* cleanup

* tweaks

* exclude no only matches

* add to release notes

* whitespace

* updating release notes

* Minor fixes (#3283)

* docs tweaks

* documenting cache=True feature

---------

Co-authored-by: manivoxel51 <mani@voxel51@gmail.com>
Co-authored-by: Lanny W <lanzhenwang9@gmail.com>
Co-authored-by: manivoxel51 <109545780+manivoxel51@users.noreply.github.com>
Co-authored-by: Brian Moore <brian@voxel51.com>
Co-authored-by: imanjra <ibrahim@voxel51.com>
Co-authored-by: topher <topher@voxel51.com>
Co-authored-by: Jacob Marks <jamarks13@gmail.com>
Co-authored-by: brimoor <brimoor@umich.edu>
Co-authored-by: Ayush Chaurasia <ayush.chaurarsia@gmail.com>
Co-authored-by: Kishan Savant <ksavant577@gmail.com>
Co-authored-by: Kacey <kacey@voxel51.com>
Co-authored-by: Kishan Savant <66986430+NeoKish@users.noreply.github.com>

* Updating logo wall links (#3288)

* updating logo wall links

* better links

---------

Co-authored-by: brimoor <brimoor@umich.edu>
Co-authored-by: Kishan Savant <ksavant577@gmail.com>
Co-authored-by: Brian Moore <brian@voxel51.com>
Co-authored-by: Kacey <kacey@voxel51.com>
Co-authored-by: topher <topher@voxel51.com>
Co-authored-by: Kishan Savant <66986430+NeoKish@users.noreply.github.com>
Co-authored-by: manivoxel51 <mani@voxel51@gmail.com>
Co-authored-by: Lanny W <lanzhenwang9@gmail.com>
Co-authored-by: manivoxel51 <109545780+manivoxel51@users.noreply.github.com>
Co-authored-by: imanjra <ibrahim@voxel51.com>
Co-authored-by: Jacob Marks <jamarks13@gmail.com>
Co-authored-by: Ayush Chaurasia <ayush.chaurarsia@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants