Skip to content

Commit

Permalink
Mute most alerts (#74)
Browse files Browse the repository at this point in the history
This PR mutes most alerts currently raised in the repo:
- All alerts from the quote test are removed.
- All alerts form the reference solver test are removed.
  • Loading branch information
fhenneke authored Oct 9, 2023
1 parent e125f1b commit f6128af
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/monitoring_tests/partially_fillable_fee_quote_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ def run_quote_test(self, trade: Trade, transaction: TxData) -> bool:
]

if abs(diff_fee_rel) > FEE_RELATIVE_DEVIATION_FLAG:
self.alert("\t".join(log_output))
self.logger.info("\t".join(log_output))
elif abs(diff_fee_rel) > FEE_RELATIVE_DEVIATION_FLAG / 10:
self.logger.info("\t".join(log_output))
else:
Expand Down
2 changes: 1 addition & 1 deletion src/monitoring_tests/reference_solver_surplus_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ def compare_orders_surplus(
a_abs_eth > SURPLUS_ABSOLUTE_DEVIATION_ETH
and a_rel > SURPLUS_REL_DEVIATION
):
self.alert(log_output)
self.logger.info(log_output)
self.logger.info(ref_solver_log)
elif (
a_abs_eth > SURPLUS_ABSOLUTE_DEVIATION_ETH / 10
Expand Down

0 comments on commit f6128af

Please sign in to comment.