efficient_return raises an unnecessary value error in the long/short case

[jgerlach93]

Describe the bug when passing a vector of expected returns with possibly negative values, as desired for the long short case, i guess max_return_value is still wrongly computed as the max(expected_returns), but, should be computed as max(weight_bounds[0] neg_expected_returns) and max(weight_bounds[1]) pos_expected_returns) for the differently signed parts of the expected_returns vector

    399 if target_return > self._max_return_value:
--> 400     raise ValueError(
    401         "target_return must be lower than the maximum possible return"
    402     )
    404 update_existing_parameter = self.is_parameter_defined("target_return")
    405 if update_existing_parameter:

ValueError: target_return must be lower than the maximum possible return

Expected behavior A clear and concise description of what you expected to happen.

Code sample Add a minimal reproducible example (see here).

Operating system, python version, PyPortfolioOpt version e.g MacOS 10.146, python 3.7.3, PyPortfolioOpt 1.2.6

Additional context Add any other context about the problem here.