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.

Sign up for GitHub

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

Jump to bottom

wee_reports does not honor new types added by extensions #95

Closed
tkeffer opened this issue Jan 17, 2016 · 1 comment
Closed

wee_reports does not honor new types added by extensions #95

tkeffer opened this issue Jan 17, 2016 · 1 comment

Comments

@tkeffer
Copy link
Contributor

tkeffer commented Jan 17, 2016

Because wee_reports does not load extensions, any additional types defined by them (by adding to obs_group_dict) are not present. The unit mappings will not be correct.

One can always add unit extensions to user/extensions.py, but this cannot easily be done by an extension writer.
tkeffer added a commit that referenced this issue Nov 6, 2019
@tkeffer
Instantiate a dummy engine. Fixes issue #95.
375338d
tkeffer added a commit that referenced this issue Nov 6, 2019
@tkeffer
Document fix for issue #95
254cc9e
@tkeffer
Copy link
Contributor Author

tkeffer commented Nov 17, 2019

Fixed in WeeWX V4

@tkeffer tkeffer closed this as completed Nov 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tkeffer