This reverts commit 5779109e.

ci: added keepLegacyBugs for some old inputs, and
deleted fixLegacyBugs as it's no longer needed because
it's the default now.

CollectBrakets now supports implicit identity operator in brakets.

ci 250 collects brakets now.

TargetingCVE reverted to use currentTimeStep from AOE,
but does advancing without using AOE otherwise.

TargetingCVE sets AOE->targetVectorsBase->time =
currentTimeStep from AOE, so printing "time" works as
expected.

TargetVectorsBase is no longer abstract.

AOE now allocates targetingVectorsBase pointer
always from its constructor.

The 3rd item of a finite loop must be separated
with colons when using more than one.
For example
FiniteLoops=[[7, 100, 0],
[-14, 100, @randomguess:tol=1e-8:mMin=100],
[ 14, 100, @save]];

tol=value,mMin=value
has been implemented for 3rd number of triplet

The 3rd number of FiniteLoop's triplet can
now be a comma-separated list; don't leave spaces
Each item in the list can be of the form
@name
or
name=value
Supported @names are
save, onlyfastwft, onlyslowwft, randomguess, multisitepush,
or onesitetruncation.

Supported names are tolerance or tol and mmin.
Names are case insensitive.
name=value has no effect for now.

TSPTau= and TSPMu= must appear in the
input file for this calcMu diagonal in site term
to take effect.

When a recovery file contains time vectors,
and is restarted into correction vector,
DMRG++ must not set the physical time,
because it's not needed or possible.

Alberto authored Jan 25, 2022 and Alvarez, Gonzalo committed Feb 02, 2022

About borders, measurements have been
fixed for TargetingExpression and TargetingTimeSteps

Now magnetic fields in the x and z direction
can both be specified independently.
MagneticFieldX and MagneticFieldZ