Fix Monte Carlo Least Core error when n_iterations < len(dataset)

[AnesBenmerzoug]

Description

This PR closes #251

Changes

• Prints a warning instead of raising an error when n_iterations < len(dataset) in Monte Carlo Least Core.
• Moved montecarlo_least_core higher in the module for readability.
• Removed test that is no longer needed.

Checklist

• Wrote Unit tests (if necessary)
• Updated Documentation (if necessary)
• Updated Changelog
• If notebooks were added/changed, added boilerplate cells are tagged with "nbsphinx":"hidden"

[mdbenito]

lgtm, but looking at this, the name n_iterations doesn't really make much sense, does it? It's the number of constraints used in the problem. Before, it made sense to use n_iterations everywhere for consistency, but now the main MC methods use StoppingCriterion. And since LC and GT need a dedicated parameter, which in both cases is a number of constraints or samples to take, maybe we should change the name to be n_samples or n_constraints in both. (?)

[AnesBenmerzoug]

I wanted to tackle the issue of consistency in the argument names in another issue.

I think we have to agree first on what to use and then document it before making changes.

[mdbenito]

I wanted to tackle the issue of consistency in the argument names in another issue.

Created #295