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
Alternative names for these permutations are "caterpillar randomization" and "cyclic shift permutation". The latter is the much more common terminology applied to these cyclic permutations.
A pity, because "caterpillar randomization" is rather charming 🐛 🕙 🐛
The suggested fix is to simply add an alias (e.g. cyclic_shift()) to replace temporal_torus_translation()
The text was updated successfully, but these errors were encountered:
Are we going to use an alias for this? I believe it was suggested to avoid breaking any existing code that used temporal_torus_translation() -- but it looks like we've just decided to change the name to cyclic_shift() everywhere. I think that's fine! I suggest we close this one.
speaking of which, are we going to rename the .R file cyclic_shift.R or keep the old name?
reviewer comment:
A pity, because "caterpillar randomization" is rather charming 🐛 🕙 🐛
The suggested fix is to simply add an alias (e.g.
cyclic_shift()
) to replacetemporal_torus_translation()
The text was updated successfully, but these errors were encountered: