Breaking: `reset_chain`->`reset_chains`, `acceptance_ratio`->`acceptance` (!677) · Merge requests · Vicentini Filippo / netket

Vicentini Filippo requested to merge breakingstuff into master May 04, 2021

Created by: PhilipVinc

From feedback from a few users:

reset_chain was singular, while everything else was plural, so let's be consistent.
SampelrState.acceptance_ratio returned the percentage (not a ratio). Also in much code that automatically adjust number of samples you need a ratio not a percentage. So fixes the bug and rename the variable to acceptance.

There are deprecation warnings in case anybody was using this stuff.

Breaking: `reset_chain`->`reset_chains`, `acceptance_ratio`->`acceptance`