Skip to content
Snippets Groups Projects

Asset range option

Merged Michał Kudela requested to merge kudmich/add_asset_range_option into master
All threads resolved!
@@ -213,7 +213,7 @@ class Asset:
lower_limit: AssetLimitT,
upper_limit: Optional[AssetLimitT] = None,
*,
tolerance: Optional[int] = None,
tolerance: Union[int, float, None] = None,
):
if not upper_limit and not tolerance:
raise TypeError("Range has to be specified with either `upper_limit` or `tolerance`")
@@ -221,9 +221,8 @@ class Asset:
if upper_limit and tolerance:
raise TypeError("Please choose only one option from `upper_limit` or `tolerance`")
if isinstance(tolerance, (int, float)):
if tolerance < 0:
raise TypeError("`tolerance` should be given as an positive number")
if tolerance and tolerance < 0:
raise TypeError("`tolerance` should be given as an positive number")
self.__lower_limit = lower_limit if upper_limit else lower_limit - (lower_limit * tolerance / 100)
self.__upper_limit = upper_limit if upper_limit else lower_limit + (lower_limit * tolerance / 100)
Loading