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

CVAT Annotate attribute documentation update #1973

Merged

Conversation

geoffrp
Copy link
Contributor

@geoffrp geoffrp commented Jul 29, 2022

What changes are proposed in this pull request?

Small update to annotate documentation to close #1748

How is this patch tested? If it is not, please explain why.

No tests are needed to verify documentation updates

Release Notes

Is this a user-facing change that should be mentioned in the release notes?

  • 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
  • [x ] Documentation: FiftyOne documentation changes
  • Other

@geoffrp geoffrp changed the title Added clarification to tie attribute argument to label_schema CVAT Annotate attribute documentation update Jul 29, 2022
@ehofesmann ehofesmann self-assigned this Aug 5, 2022
@ehofesmann ehofesmann added documentation Documentation work annotation Issues related to FiftyOne's annotation API labels Aug 5, 2022
@ehofesmann ehofesmann self-requested a review August 5, 2022 21:22
Copy link
Member

@ehofesmann ehofesmann left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! This is a nice clarification!

@ehofesmann ehofesmann merged commit 22ba8c1 into voxel51:develop Aug 8, 2022
kaixi-wang pushed a commit that referenced this pull request May 11, 2023
* bson fix

* dataset 404s

* fix selection css and setting

* bumps

* release notes

* bump teams app

* 404 fix

* bump os compat

* Fix custom_parser implementation (get_label should return an instance of the label_cls)

* bump helm charts for 0.1.7 (#79)

* more get_field() fixes

* adding support for serializing aggregations

* adding serialization tests for aggregations

* linting

* adding __eq__ methods

* linting

* more unit testy

* bug fixes

* adding set_values() and exclude_fields() tests

* adding get_field() tests

* adding label ID tests

* reverting to v0.15.1 implementation

* fixing #1893

* id updates

* adding id back to repr

* fixing more ID bugs

* adding rel_dir option

* finishing implementation

* Fix bug when loading group ids in CVAT video tasks (#1917)

* catch tags in app aggregation data flow (#1924)

* adding weighted_sample() and balanced_sample() utils

* moving random_split() to random module

* updating docstrings

* bug

* adding new_ids option to add_collection()

* adding tests for add_collection()

* Resolve bug when uploading to projects in CVAT (#1926)

* proper handling of db fields in mixin methods

* proper handling of db fields in view stages/saving

* fixing clips media type bug

* removing duplicate method definition

* fixing edge case and adding tests

* fixing typo

* view refresh fix

* fix result alignment

* adding quantile aggregation

* adding unit tests for quantile aggregation

* bug fix

* handling string vs numeric

* adding bad values test

* adding quantiles to docs

* implementing kwargs

* relaxing numpy version requirement

* updating nan tests

* supporting serializing and deserializing of arbitrary ObjectId data

* handling SON serialization

* adding stats() method to sample collections

* move all auth0 Teams App configuration to env config

* add targets conversion to graphql datasets (#1943)

* use correct dict default

* fixing bug

* db field is no longer necessary

* fixing one part of #1945

* fixing #1945

* adding object ID tests

* removing default_classes usage

* removing outdated syntax

* adding missing docs

* raising informative error when trying to combine splits

* fixing frames bug

* testing ID fields/db fields

* make ObjectIdField handle str/ObjectId conversions

* validating sample_id field

* documenting stats() method

* tweaking langugage

* more robust implementation

* optimizing convert datasets method

* adding test to cover expected ObjectId behavior

* fixing ObjectId bugs in to_dict()/from_dict()

* unnecessary

* fixing set indexing bug

* unique

* only use patches selection mode for spatial label fields

* fixing bool bugs

* adding bool unit test

* adding numpy bool support

* added field name validation

* finishing implementation of ObjectIdField handling for DatasetViews

* fixing bug

* handling edge case

* proper handling of private fields when serializing collections

* adding dataset serialization tests

* updating unit test

* adding dynamic/BSON field tests

* tweaking todos

* fixing tests

* adding persistent option to clone()

* clarifying that add field methods are idempotent

* production defaults

* be more explicit

* removes helm chart from fiftyone-teams repo(#81)

* helm charts moved to fiftyone-teams-app-deploy repo

* removes deployment/helm from .gitignore

* removes helm chart from fiftyone-teams repo(#81)

* helm charts moved to fiftyone-teams-app-deploy repo

* removes deployment/helm from .gitignore

* adding missing brain call

* fix session views

* gracefully casting to numpy array

* updating documentation

* lifting plotly<5 requirement

* adding missing brain call

* update tabulate to 0.8.10 for source install

* Added clarification to tie attribute argument to label_schema (#1973)

* trying looser requirements

* making tags optional

* fixing bug

* documenting explanation of #1748 in all relevant places

* passing attributes through during label coercions

* linting

* fixing test

* adding field_names, iter_fields(), and merge() to SerializableDocument

* CVAT optimizations (#1944)

* avoid listing tasks at download time

* pass api into results methods

* document api optimization

* lint

* lint pro tip

* add api close context

* refactoring connect_to_api() into an interface concept

* refactoring connect_to_api() into an interface concept

* updating documentation

* cleanup

* moving context manager to AnnotationAPI class

Co-authored-by: brimoor <brimoor@umich.edu>
Co-authored-by: Brian Moore <brian@voxel51.com>

* noting that dataset-level metadata is excluded

* Label studio integration (#1848)

* add label studio integration for classification, detection and polyline

* handle export and import of other label types

* upload predictions to label studio for classification

* upload predictions for all label types

* update labelstudio docstrings and formatting

* store existing labels in id_map

* read api key from env or prompt, small fixes

* add supports attrs property

* add label studio integration for classification, detection and polyline

* handle export and import of other label types

* upload predictions to label studio for classification

* upload predictions for all label types

* update labelstudio docstrings and formatting

* store existing labels in id_map

* read api key from env or prompt, small fixes

* raise error if label studio version is below 1.5,
prompt for a specific label studio sdk version if not installed

* add warning about ignoring attributes

* add labelstudio docs

* fix tests

Co-authored-by: Eric Hofesmann <ehofesmann@gmail.com>

* adding support for writing transformed images to a new location

* adding support for writing transformed videos to a new location

* adding unique filename utils

* updating CLI

* Label studio updates (#2006)

* add label studio integration for classification, detection and polyline

* handle export and import of other label types

* upload predictions to label studio for classification

* upload predictions for all label types

* update labelstudio docstrings and formatting

* store existing labels in id_map

* read api key from env or prompt, small fixes

* add supports attrs property

* add label studio integration for classification, detection and polyline

* handle export and import of other label types

* upload predictions to label studio for classification

* upload predictions for all label types

* update labelstudio docstrings and formatting

* store existing labels in id_map

* read api key from env or prompt, small fixes

* raise error if label studio version is below 1.5,
prompt for a specific label studio sdk version if not installed

* add warning about ignoring attributes

* add labelstudio docs

* fix tests

* update docstrings

* add supports video flag to annotation backend

* update connect_to_api

* parse config parameters for labelstudio tests

* track label ids to merge properly

* support unsubmitted tasks

* note supported scalar types

* currently doesnt support classifications plural

* linting

* fixing typos

* linting docs

* linting

* one more linting pass

* final pass

* fix labelstudio tests

Co-authored-by: rusteam <rustemgal@gmail.com>
Co-authored-by: brimoor <brimoor@umich.edu>

* bumps, release notes

* add to release notes

* linting

* converting to public SaveContext class

* adding missing _save_replacements implementation

* adding public save_context() method

* adding logging configuration

* documenting

* finishing work

* generated views don't support save contexts

* setting default behavior

* compute ops in real-time

* adding ipywidgets<8 requirement

* documenting save contexts

* refactoring into a deferred=True option

* fixing docs warnings

* adding unit tests

* unnecessary

* adding more examples, standardizing default batch_size logic

* adding compatible DB versions

* using client version terminology

* making DatasetDocument dynamic

* optimizing get dataset version

* read path variable from dataset.yaml

* update docs for path in dataset.yaml

* tweaking docs

* tweaks

* avoid deserializing extra fields

* updating unit test

* using strict=False in more places

* more non-strict

* showing available logging levels

* adding normpath

* updating compatibility version

* documenting compatible versions

* fixing LegacyFiftyOneDataset import bug

* removes artifacts I should not have added (#88)

* clarifying

* oops

* updating pkg versions

* removing unnecessary pull

* removes helm chart from fiftyone-teams repo(#81)

* helm charts moved to fiftyone-teams-app-deploy repo

* removes deployment/helm from .gitignore

* removes artifacts I should not have added (#88)

* remove editable flag

* move App build to end

* updating release notes

* always migrate when user is admin

* moving legacy troubleshooting to another page

* adding docs on coordinating migrations

* adding helpful error when a dataset fails to load

* pinning max requirement

* using more cloud-friendly layout

* cloud-friendly updates

* adding upgrade note

* reverting premature version changes

* packages bumps

Co-authored-by: Benjamin Kane <ben@voxel51.com>
Co-authored-by: brimoor <brimoor@umich.edu>
Co-authored-by: Brian Moore <brian@voxel51.com>
Co-authored-by: idow09 <idow09@gmail.com>
Co-authored-by: Eric Hofesmann <ehofesmann@gmail.com>
Co-authored-by: imanjra <ibrahim@voxel51.com>
Co-authored-by: Geoffrey Keating <96071142+geoffrp@users.noreply.github.com>
Co-authored-by: Rustem Galiullin <rustemgal@gmail.com>
Co-authored-by: Victor Oancea <victor.oancea@me.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
annotation Issues related to FiftyOne's annotation API documentation Documentation work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Annotation schema attribute definitions not filtering when defined on classes
2 participants