You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
With the introduction of Payload with modalities, we have multiple payloads in the datapack. Therefore, it's not good to continue to use DataPack.text which could be ambiguous in which text payload it refers to. In the past, DataPack.text refers to the single text in the DataPack, and for backward compatibility we let it be the text in the first TextPayload.
To Reproduce
Steps to reproduce the behavior:
Prepare data ...
Run command '....'
See error
Expected behavior
DataPack.text is removed
All code dependent on it will be changed to use DataPack.get_payload_data_at(Modality.Text, 0)
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):
OS: [e.g. iOS]
Version [e.g. 22]
Python and Package verions: [e.g. Python version, Pytorch version]
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
hmm, sorry I didn't notice that one. Yes, let's add it back. Now is not the time to change the interfaces. Instead we should make sure all old interfaces work.
Describe the bug
With the introduction of Payload with modalities, we have multiple payloads in the datapack. Therefore, it's not good to continue to use
DataPack.text
which could be ambiguous in which text payload it refers to. In the past,DataPack.text
refers to the single text in the DataPack, and for backward compatibility we let it be the text in the first TextPayload.To Reproduce
Steps to reproduce the behavior:
...
Expected behavior
DataPack.text
is removedDataPack.get_payload_data_at(Modality.Text, 0)
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: