UPDATE:
Я в корне неправильно понял проблему. Феликс запрашивал у mongoDB, сколько предметов попало в каждый диапазон; поэтому мой подход не сработал, потому что я пытался попросить mongoDB для пунктов. У Феликса много данных, поэтому это совершенно неразумно.
Феликс, вот обновленная функция, которая должна делать то, что вы хотите:
def getDataFromLast(num, quantum):
m = my_mongodb()
all = []
not_deleted = []
today = datetime.combine(date.today(), time())
for i in range(num+1)[-1]: # start from oldest
day = today - i*quantum
time_query = {"$gte":day, "$lt": day+quantum}
all.extend(m.data.find({"time":time_query}).count())
not_deleted.extend(m.data.find({"deleted":0, "time":time_query}).count())
return all, not_deleted
Квант - это «шаг», чтобы оглянуться назад. Например, если мы хотим посмотреть на последний
12 часов я бы установил quantum = timedelta(hours=1) и num = 12.
Обновленный пример использования, где мы получаем последние 30 дней, будет:
from datetime import datetime, date, time, timedelta
import matplotlib.pyplot as plt
import matplotlib.ticker as ticker
from my_conn import my_mongodb
#def getDataFromLast(num, quantum) as defined above
def format_date(x, N, pos=None):
""" This is your format_date function. It now takes N
(I still don't really understand what it is, though)
as an argument instead of assuming that it's a global."""
day = date.today() - timedelta(days=N-x-1)
return day.strftime('%m%d')
def plotBar(data, color):
plt.bar(range(len(data)), data, align='center', color=color)
N = 30 # define the range that we want to look at
all, valid = getDataFromLast(N, timedelta(days=1)) # get the data
plotBar(all, "#4788d2") # plot both deleted and non-deleted data
plotBar(valid, "#0c3688") # plot only the valid data
plt.xticks(range(N), [format_date(i) for i in range(N)], size='small', rotation=30)
plt.grid(axis="y")
plt.show()
Оригинал:
Хорошо, это моя попытка рефакторинга для вас. Blubber предложил изучить JS и MapReduce. В этом нет необходимости, если вы будете следовать его другим советам: создать индекс для поля времени и сократить количество запросов. Это моя лучшая попытка в этом, наряду с небольшим рефакторингом. У меня есть куча вопросов и комментариев, хотя.
Начиная с:
with my_mongodb() as m:
for i in range(30):
day = today - timedelta(days = i)
t1 = [m.data.find({"time": {"$gte": day, "$lt": day + timedelta(days = 1)}}).count()] + t1
t2 = [m.data.find({"deleted": 0, "time": {"$gte": day, "$lt": day + timedelta(days = 1)}}).count()] + t2
Вы делаете запрос mongoDB, чтобы найти все данные за каждый день за последние 30 дней. Почему бы вам просто не использовать один запрос? И когда у вас есть все данные, почему бы просто не отфильтровать удаленные данные?
with my_mongodb() as m:
today = date.today() # not sure why you were combining this with time(). It's the datetime representation of the current time.time()
start_date = today -timedelta(days=30)
t1 = m.find({"time": {"$gte":start_date}}) # all data since start_date (30 days ago)
t2 = filter(lambda x: x['deleted'] == 0, all_data) # all data since start_date that isn't deleted
Я действительно не уверен, почему вы делали 60 запросов (30 * 2, один для всех данных, один для не удаленных). Есть ли какая-то конкретная причина, по которой вы собирали данные изо дня в день?
Тогда у вас есть:
x = range(30)
N = len(x)
Почему бы и нет:
N = 30
x = range(N)
len(range(x) равно x, но требует времени для вычисления. То, как вы это написали изначально, просто немного ... странно.
Вот мой недостаток, с предложенными мною изменениями, как можно более общими.
from datetime import datetime, date, time, timedelta
import matplotlib.pyplot as plt
import matplotlib.ticker as ticker
from my_conn import my_mongodb
def getDataFromLast(delta):
""" Delta is a timedelta for however long ago you want to look
back. For instance, to find everything within the last month,
delta should = timedelta(days=30). Last hour? timedelta(hours=1)."""
m = my_mongodb() # what exactly is this? hopefully I'm using it correctly.
today = date.today() # was there a reason you didn't use this originally?
start_date = today - delta
all_data = m.data.find({"time": {"$gte": start_date}})
valid_data = filter(lambda x: x['deleted'] == 0, all) # all data that isn't deleted
return all_data, valid_data
def format_date(x, N, pos=None):
""" This is your format_date function. It now takes N
(I still don't really understand what it is, though)
as an argument instead of assuming that it's a global."""
day = date.today() - timedelta(days=N-x-1)
return day.strftime('%m%d')
def plotBar(data, color):
plt.bar(range(len(data)), data, align='center', color=color)
N = 30 # define the range that we want to look at
all, valid = getDataFromLast(timedelta(days=N))
plotBar(all, "#4788d2") # plot both deleted and non-deleted data
plotBar(valid, "#0c3688") # plot only the valid data
plt.xticks(range(N), [format_date(i) for i in range(N)], size='small', rotation=30)
plt.grid(axis="y")
plt.show()