Add compose parallel rollout

[jasonkhadka]

• Added compose, parallel, rollout, namedrollout, namedparallel
• implemented __call__ for these
• renamed MetaTransforms to CompoundTransforms
• resolved inheritance issue - all inherit from CompoundTransforms
• unresolved - module and stack arguments for __init__

[sjrl]

It looks like some cpython files were added accidentally. Could you remove them and add __pycache__ to the .gitignore?

[sjrl]

Looking nice and clean!

[jasonkhadka]

It looks like some cpython files were added accidentally. Could you remove them and add __pycache__ to the .gitignore?

removed the files.

[sjrl]

Is it possible to add tests for these functions? Or are they not usable yet?

[jasonkhadka]

Is it possible to add tests for these functions? Or are they not usable yet?

Yup not usable yet without figuring out module and stack. I guess that has to be inferred either inside CompoudTransform or inside individual Transforms.

[jasonkhadka]

I see an issue with flatten for NamedRollout and NamedParallel, flattening the named entity would result in change of keys and format of namedtuple output. Named* need to have custom _flatten_list
Also should this be the desired result of flattening? What about the key t2?
NamedRollout(t1=t1, t2=(NamedRollout(t2a=t2a,t2b=t2b)) = NamedRollout(t1=t1, t2a=t2a, t2b=t2b)

[wuhu]

I see an issue with flatten for NamedRollout and NamedParallel, flattening the named entity would result in change of keys and format of namedtuple output. Named* need to have custom _flatten_list Also should this be the desired result of flattening? What about the key t2? NamedRollout(t1=t1, t2=(NamedRollout(t2a=t2a,t2b=t2b)) = NamedRollout(t1=t1, t2a=t2a, t2b=t2b)

hmm it would seem logical to me not to flat those

[sjrl]

I see an issue with flatten for NamedRollout and NamedParallel, flattening the named entity would result in change of keys and format of namedtuple output. Named* need to have custom _flatten_list Also should this be the desired result of flattening? What about the key t2? NamedRollout(t1=t1, t2=(NamedRollout(t2a=t2a,t2b=t2b)) = NamedRollout(t1=t1, t2a=t2a, t2b=t2b)

hmm it would seem logical to me not to flat those

Does that mean flattening would not work for NamedTuple outputs?

[jasonkhadka]

I see an issue with flatten for NamedRollout and NamedParallel, flattening the named entity would result in change of keys and format of namedtuple output. Named* need to have custom _flatten_list Also should this be the desired result of flattening? What about the key t2? NamedRollout(t1=t1, t2=(NamedRollout(t2a=t2a,t2b=t2b)) = NamedRollout(t1=t1, t2a=t2a, t2b=t2b)

hmm it would seem logical to me not to flat those

This would mean that we would need to assert flatten=False for Named stuff, or override _flatten_list to do nothing?
Was thinking if other non-named entities passed to Named* could be still be flattened but i think it also does not make sense.
Example :
NamedRollout(t1=t1, t2=NamedRollout(t2a=t2a,t2b=t2b), t3=(t3a / t3b)) = ?