unitxt icon indicating copy to clipboard operation
unitxt copied to clipboard

Wrong naming and compositions

Open elronbandel opened this issue 1 year ago • 2 comments

Currently in unitxt there are wrong names and compositions for few importatnt core components in unitxt

Main Classes

  • TaskCard -> DataToTask or DataToTaskCard preprocess_steps -> data_to_task_steps
  • FormTask -> Task

Base Operators

StreamInstanceOperator -> InstanceOperator

Operators

CopyField -> Copy(field="x", to_field="y") # and remove field_to_field

elronbandel avatar May 16 '24 13:05 elronbandel

can i work on this? // @elronbandel

duckling69 avatar May 23 '24 15:05 duckling69

@duckling69 it will be great if you can work on the operators! (the rest is half done half in progress).

We need to go over the operators and make sure there is no redundancy in the name, basically removing the word Field from many operators. Also we need to make sure that at least for now, the old names are supported. So if you change CopyFIeld -> Copy you need to create:

class CopyField(Copy)
     pass

elronbandel avatar May 27 '24 07:05 elronbandel