wlanslovenija / PiplMesh

A social info portal for wireless networks.
http://dev.wlan-si.net/wiki/PiplMesh
Other
40 stars 19 forks source link

weather panel #292

Closed dcrystalj closed 11 years ago

dcrystalj commented 11 years ago

https://github.com/wlanslovenija/PiplMesh/issues/210

dcrystalj commented 11 years ago

please take a look

dcrystalj commented 11 years ago

please take a look

mitar commented 11 years ago

What is current state with other errors you reported? DuplicateKeyError? inheritance? null values? I am not sure what exactly is the problem with null value and at?

dcrystalj commented 11 years ago

here is error log on unique fields http://www.rubik.si/klub/dke.htm

dcrystalj commented 11 years ago

please take a look to last commit, also if design is ok to you...

dcrystalj commented 11 years ago

updated

mitar commented 11 years ago

So, it looks good. Those queries are a bit strange, but we can play with that later. Maybe really add TODO. (Maybe for me, to check it again when it will be pulled in.)

We should only really solve the rate limiting problem.

dcrystalj commented 11 years ago

updated

mitar commented 11 years ago

To make less entries stored: store only predictions for future three days. But make this a top-level Python constant (for how many days data should be fetched and stored).

dcrystalj commented 11 years ago

updated

mitar commented 11 years ago

Please update from main repository.

mitar commented 11 years ago

We had to change way periodic tasks are run because they were piling up. Sorry for additional work. So now we do not define periodic tasks in settings.py anymore.

mitar commented 11 years ago

Otherwise it is great now! Super!

dcrystalj commented 11 years ago

bump

mitar commented 11 years ago

Otherwise it looks good. But we should be doing tests too. Check how I made tests for horoscope. You can make them, or open a ticket for them. You have done a lot already.

dcrystalj commented 11 years ago

updated, still some bug in test(frozen) but i think it's not a problem in code

mitar commented 11 years ago

Merged. Tests to be done with #326.

mitar commented 11 years ago

It seems tests are still failing, there is still bug in the code. Grr. You should really try your code before pushing!

https://travis-ci.org/#!/wlanslovenija/PiplMesh/builds/2686585

dcrystalj commented 11 years ago

maybe you haven't seen my comment https://github.com/wlanslovenija/PiplMesh/commit/e6c61828bfd883e438662d1a75652942b9692a9b i have problems again to run/install requirements