Removed useless comments + prevent trying to add an already present element in the set

This commit is contained in:
Mokaddem 2016-07-26 11:06:31 +02:00
parent cc1b49baaf
commit baa042827d
2 changed files with 10 additions and 14 deletions

View file

@ -47,6 +47,9 @@ def compute_most_posted(server, message, num_day):
curr_value = server.hget(date, module+'-'+keyword) curr_value = server.hget(date, module+'-'+keyword)
keyword_total_sum += int(curr_value) if curr_value is not None else 0 keyword_total_sum += int(curr_value) if curr_value is not None else 0
if keyword in server.smembers(redis_progression_name_set): # if it is already in the set
return
if (server.scard(redis_progression_name_set) < max_set_cardinality): if (server.scard(redis_progression_name_set) < max_set_cardinality):
server.sadd(redis_progression_name_set, keyword) server.sadd(redis_progression_name_set, keyword)
@ -91,25 +94,19 @@ def compute_provider_size(server, path, num_day_to_look):
server.hset(paste_provider+'_num', paste_date, 1) server.hset(paste_provider+'_num', paste_date, 1)
# Compute Most Posted # Compute Most Posted
#date_range = get_date_range(num_day_to_look)
# check if this keyword is eligible for progression # check if this keyword is eligible for progression
provider_total_sum = 0
#for date in date_range:
# curr_value = server.hget(paste_provider+'_size', date)
# provider_total_sum += int(curr_value) if curr_value is not None else 0
#if paste_provider in server.smembers(redis_progression_name_set): # if it is already in the set if paste_provider in server.smembers(redis_progression_name_set): # if it is already in the set
# return return
if (server.scard(redis_progression_name_set) < max_set_cardinality): elif (server.scard(redis_progression_name_set) < max_set_cardinality):
server.sadd(redis_progression_name_set, paste_provider) server.sadd(redis_progression_name_set, paste_provider)
else: #not in the set else: #set full capacity
#Check value for all members #Check value for all members
member_set = [] member_set = []
for provider in server.smembers(redis_progression_name_set): for provider in server.smembers(redis_progression_name_set):
curr_avg = 0.0 curr_avg = 0.0
# for date in date_range:
curr_size = server.hget(provider+'_size', paste_date) curr_size = server.hget(provider+'_size', paste_date)
curr_num = server.hget(provider+'_num', paste_date) curr_num = server.hget(provider+'_num', paste_date)
if (curr_size is not None) and (curr_num is not None): if (curr_size is not None) and (curr_num is not None):
@ -120,7 +117,6 @@ def compute_provider_size(server, path, num_day_to_look):
#remove min from set and add the new one #remove min from set and add the new one
print 'Adding ' +paste_provider+ '(' +str(new_avg)+') in set and removing '+member_set[0][0]+'('+str(member_set[0][1])+')' print 'Adding ' +paste_provider+ '(' +str(new_avg)+') in set and removing '+member_set[0][0]+'('+str(member_set[0][1])+')'
server.srem(redis_progression_name_set, member_set[0][0]) server.srem(redis_progression_name_set, member_set[0][0])
server.srem(redis_progression_name_set, member_set[0][0])
server.sadd(redis_progression_name_set, paste_provider) server.sadd(redis_progression_name_set, paste_provider)

View file

@ -4,7 +4,7 @@
<div class="col-lg-12"> <div class="col-lg-12">
<div class="panel panel-default"> <div class="panel panel-default">
<div class="panel-heading"> <div class="panel-heading">
<i id="flash-tld" class="fa fa-unlock" flash-tld=""></i> <strong> Credential</strong> - Most posted domain <i id="flash-tld" class="fa fa-unlock" flash-tld=""></i> <strong> Credential</strong> - Today most posted domain
</div> </div>
<div class="panel-body"> <div class="panel-body">
<div class=""> <div class="">
@ -28,7 +28,7 @@
<div class="col-lg-12"> <div class="col-lg-12">
<div class="panel panel-default"> <div class="panel panel-default">
<div class="panel-heading"> <div class="panel-heading">
<i id="flash-tld" class="fa fa-envelope" flash-tld=""></i><strong> Mail</strong> - Most posted domain (max 1 per paste) <i id="flash-tld" class="fa fa-envelope" flash-tld=""></i><strong> Mail</strong> - Today most posted domain (max 1 per paste)
</div> </div>
<div class="panel-body"> <div class="panel-body">
<div class=""> <div class="">
@ -52,7 +52,7 @@
<div class="col-lg-12"> <div class="col-lg-12">
<div class="panel panel-default"> <div class="panel panel-default">
<div class="panel-heading"> <div class="panel-heading">
<i id="flash-size" class="fa fa-database" flash-tld=""></i><strong> Paste size</strong> - by paste provider <i id="flash-size" class="glyphicon glyphicon-transfer" flash-tld=""></i><strong> Provider</strong> - Today average paste size
</div> </div>
<div class="panel-body"> <div class="panel-body">
<div class=""> <div class="">