Trends Chart

[pstoica]

Maybe try to make this line up with the months in the calendar view?

[camreon]

will try to update the design of this chart before class. it's filling in under the line for some reason and looks rough. might have time to add chart for #2 also

[pstoica]

think you're missing some css (stylesheets/main.less), looked at http://bl.ocks.org/mbostock/3883245

.line {
  fill: none;
  stroke: steelblue;
  stroke-width: 1.5px;
}

[pstoica]

pretty sure we should combine the charts into one. even though trends is on a different scale (we can rescale it), it's merely a suggestion instead of hard data so i think it should work out.

[camreon]

yeah, it was missing that. sounds good. i want to add horizontal grid lines too

[pstoica]

@camreon the price history shows up now. some to-do for the charts

• [x] prevent total redrawing
• [x] fix scale so it goes up to $35
• [ ] make the red and blue match the current ones
• [x] 'reduce chart junk' lol
• [x] cut off trends data at beginning of 2010, or there's rendering issues causing it to go left out of the axis, one of those
• [ ] descriptions for events before 2010? (unrelated)

I'll work on the linking, and maybe tooltips for the maps and the charts?

[pstoica]

also, the price chart doesn't reflect null values right now since there are no entries for those months. should we do null values, or let it bridge the gap?

[camreon]

working on showing a popup of data when you mouse over the chart now and then cleaning up the rest of the chart.

i can't figure out why the trend data is going out of the chart's bounds. do either of you want to look into that more?

and i think it's fine to let it bridge gap months for now

[pstoica]

takin a look

[pstoica]

fixed, it was still drawing dates before 2010, so i filtered that on the server side.

[pstoica]

I just added caching for all the JSON responses.

[pstoica]

fyi, the mongo mapreduce was kind of wrong. you should see more data now, actually.

[camreon]

@pstoica why are you keeping track of all the past queried prices in line 331?

prices = $.merge(prices, json.prices);

[pstoica]

I threw it Ito a global I think so I could use it for the popups on the map?

[pstoica]

*threw it into

[pstoica]

@camreon that code was actually meant to fill the prices with null values (so that there are visible gaps) and replace them with whatever isn't null. or do you want to leave it as continuous? i feel like we should show gaps so we don't mislead the user or whatever.

[camreon]

ah k. i agree, showing gaps is more honest. i was just being lazy before lol