Updated heatmaps to pull random data from server

prashantchhabra89 / Alternate-Power-Source-Property-Mapper

A web application that puts the power into people's hands for finding sites for sustainable electricity generation.

http://power-planner.appspot.com/

Apache License 2.0

0 stars 1 forks source link

Updated heatmaps to pull random data from server #22

Closed maxentropic closed 9 years ago

maxentropic commented 9 years ago

This update has the heatmaps pulling in random data from the server through an asynchronous POST request. It's not particularly efficient, and new random data is asked for every time anything happens, but it's a working end-to-end proof of concept for heatmaps functionality.

Some important things to note:

@CYLoe I made the constructors of DataTuple public. This was just for generating fake data, so they can go back to how they were before without much trouble. Was there a specific reason they weren't public?
If you zoom out too much, the page gets very unresponsive
Client side caching will be my next priority, I think.

CYLoe commented 9 years ago

@maxentropic the reason is I was being a lazy programmer considering I wrote those codes 5 am in the morning. Just leave them as public, they were supposed to be that.

maxentropic commented 9 years ago

@prashantchhabra89 I've updated my pull request now to remove the datastore populator package. It's ready for reviewing and merging.

prashantchhabra89 commented 9 years ago

@maxentropic Nice comments! Any specific reason to have this code in comments in file PowerPlannerServlet.java Line 54-61 @param pType

* param neLat
* param neLng
* param swLat
* param swLng
* return

wow had to remove @. it tagged some users on github lol

maxentropic commented 9 years ago

Oh right, I was just adding the "Temporary method for generating fake server data" line, and eclipse auto-generated the param tags. I could delete them if you want.

prashantchhabra89 commented 9 years ago

that is ok :)