[pull] master from datasaur-ai:master#2
Open
pull[bot] wants to merge 7 commits intoivanm-gdp:masterfrom
Open
Conversation
* handles COCO <-> Datasaur Schema conversion * custom attributes handling * also: multiple project from multiple folders
to handle filenames with multiple dots. also: - fix: typings for shapes_from_coco - fix: adjust None handling for ignored_attributes
* separated some logics from Project class to new classes GraphQLUtils and GraphQLDocumentCreator * createdf ProjectInBatch class to create multiple projects based on the amount of batched documents * updated api_client for the new usage of ProjectInBatch and Project classes * Fix code smells in api_client.py, reorder imports * add type hints to object initialization and method params, change self.headers default value to empty dict * update readme to mention minimum Python version * change default document_batch_size to 20 * change private methods to protected * change default batch size to 100 * rename ProjectInBatch to BatchedProject * fix typo in log message * introduce __report_before_request * update readme, add sub-section about batched project creation * extracted default documents path to constant * moved getting access token for Header to Project.__init__ * import GraphQLDocument from graphql_document_creator * added logging when getting access token and uploading files * remove unused import * use ThreadPoolExecutor to upload multiple files at once * add logging: total project creation request that will be submitted * move upload_and_create_document to a class method * #23456 add xml to __sort_possible_extra_files_last * #23456 remove print * add xml to __sort_possible_extra_files_last * use `sorted` and extract valid extensions as a tuple --------- Co-authored-by: Ivan Marcellino <ivan.marcellino@datasaur.ai>
- adjust zipfile reading process to always use `/` - use Python's built-in module for creating temporary folder - adjustment on ids: ensure uniqueness using array length and start at 1
* Add Create Project via EOS * address reviews suggestions
* skip fetching id from custom_labelset, use array indices as unique id * intentionally set custom-label-set's id to be duplicated * also add duplicate internalId
…37) also: format export.py using Black formatter
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )