Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add summaries to historygenerator.py #11

Closed
Loughlinstown opened this issue Mar 28, 2020 · 5 comments
Closed

Add summaries to historygenerator.py #11

Loughlinstown opened this issue Mar 28, 2020 · 5 comments
Assignees

Comments

@Loughlinstown
Copy link

Thanks very much for your skin and previous help J.
I see that https://github.com/brewster76/fuzzy-archer/blob/master/bin/user/historygenerator.py file now includes a summary column (see http://meteolavall.no-ip.org/weewx/history.html for a sample page which includes it).
I really like your implementation of the NOAA monthly summaries and am not keen to lose that style of presentation.

I've tried comparing the code between your modified version of brewster76's historygenerator linked above, and this latest one and whilst a lot of it is identical I'm afraid it's beyond my abilities to correctly add the summary column functionality from one to the other.
Any chance you might review at some point and add to your modified version?

My own summary page is here if you are interested - https://www.loughlinstown.ie/records.html

Many thanks,

Muireadach.

@weatherstorm
Copy link
Owner

Thanks for using my skin. I didn't know he updated his script with a sum column. I believe merged my updated code base for the historygenerator.py before he did this update but I will look into it.

@weatherstorm
Copy link
Owner

weatherstorm commented Apr 2, 2020

I have the updated to the new version of historygeneratoy.py with my minor changes and its working on my server. If it remains stable, I will upload the code within a few days.

http://cw9009.x10host.com/summary.html

@Loughlinstown
Copy link
Author

Nice work!, thank you so much!

@Loughlinstown
Copy link
Author

Just installed the new file, works great!, thanks again.

@weatherstorm
Copy link
Owner

Thanks for letting me know about the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants