darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-26 17:24:23 -04:00
Code Issues Packages Projects Releases Wiki Activity

Don’t use single letter variable names

Browse Source 
Better to call the variables what they are.

Also re-orders the statement to avoid adding to a negative number.
This commit is contained in:
Chris Hill-Scott
2020-09-16 08:25:42 +01:00
parent ce35200453
commit 6b7908fecb
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/models/broadcast_message.py
View File

@@ -16,10 +16,10 @@ from app.notify_client.broadcast_message_api_client import (
from app.notify_client.service_api_client import service_api_client from app.notify_client.service_api_client import service_api_client
def round_to_significant_figures(x, n): def round_to_significant_figures(value, number_of_significant_figures):
return int(round( return int(round(
x, value,
-int(floor(log10(abs(x)))) + (n - 1) number_of_significant_figures - int(floor(log10(abs(value)))) - 1
)) ))