-
Notifications
You must be signed in to change notification settings - Fork 68
[PLT-1485] Removed Datatypes #1830
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
Changes from 40 commits
0bbd7c2
51ecfea
8d078c8
24e0766
2faf9a1
2c0c677
c089215
30819db
761b1e9
379171a
5e87f4e
5fc6ff3
9cf28a1
7fc10bb
8051d50
0b810fb
f8b8fa3
556a5db
aa551e1
2329324
4457f25
f5b6c7d
ce60b24
12aa8c5
f124e9d
b614ced
e31d118
1a8189a
62f5fbd
6c11e74
1ef53cc
159e227
5f625c8
90c1a19
f91a229
709ca61
fdabc94
5bb3c97
4888346
c32bd71
e0eb4c3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -40,47 +40,6 @@ def _assign_ids(label: Label): | |
self._fns["assign_feature_schema_ids"] = _assign_ids | ||
return self | ||
|
||
def add_url_to_data( | ||
self, signer: Callable[[bytes], str] | ||
) -> "LabelGenerator": | ||
""" | ||
Creates signed urls for the data | ||
Only uploads url if one doesn't already exist. | ||
|
||
Args: | ||
signer: A function that accepts bytes and returns a signed url. | ||
Returns: | ||
LabelGenerator that signs urls as data is accessed | ||
""" | ||
|
||
def _add_url_to_data(label: Label): | ||
label.add_url_to_data(signer) | ||
return label | ||
|
||
self._fns["add_url_to_data"] = _add_url_to_data | ||
return self | ||
|
||
def add_to_dataset( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same concept of add_url_to_data. This was only used in TextData and GenericDataRow does not support this method There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I can add it to GenericDataRow but I think methods like this were never used and are complicated |
||
self, dataset: "Entity.Dataset", signer: Callable[[bytes], str] | ||
) -> "LabelGenerator": | ||
""" | ||
Creates data rows from each labels data object and attaches the data to the given dataset. | ||
Updates the label's data object to have the same external_id and uid as the data row. | ||
|
||
Args: | ||
dataset: labelbox dataset object to add the new data row to | ||
signer: A function that accepts bytes and returns a signed url. | ||
Returns: | ||
LabelGenerator that updates references to the new data rows as data is accessed | ||
""" | ||
|
||
def _add_to_dataset(label: Label): | ||
label.create_data_row(dataset, signer) | ||
return label | ||
|
||
self._fns["assign_datarow_ids"] = _add_to_dataset | ||
return self | ||
|
||
def add_url_to_masks( | ||
self, signer: Callable[[bytes], str] | ||
) -> "LabelGenerator": | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,2 @@ | ||
from .audio import AudioData | ||
from .conversation import ConversationData | ||
from .dicom import DicomData | ||
from .document import DocumentData | ||
from .html import HTMLData | ||
from .raster import ImageData | ||
from .raster import MaskData | ||
from .text import TextData | ||
from .video import VideoData | ||
from .llm_prompt_response_creation import LlmPromptResponseCreationData | ||
from .llm_prompt_creation import LlmPromptCreationData | ||
from .llm_response_creation import LlmResponseCreationData | ||
from .generic_data_row_data import GenericDataRowData |
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
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.
Removed this function. This is not supported by GenericDataType and I do not think this is relevant any longer. Check doc string for what it does