I will try it, but let me explain why I came to name the aggregation type
that way.
"integral" is just the opposite of "tderiv", which is available somewhere.
So it seems to me not that special.
But why not name it just "integral"? Because you need a separate entry in
agg_group for each unit label. The principle is always the same: time is
added to the unit. But it is not possible to express that in agg_group on a
general basis.
Integration over power to get energy is not only useful for sun radiation,
but also for the measurements of solar power plants. So I saw it not that
special for that reason, too.
Tom Keffer schrieb am Dienstag, 9. März 2021 um 14:07:56 UTC+1:
> Your aggregation type energy_integral is too specialized to be included.
>
> However, I'm inclined to just get rid of the list. Can you give it a try?
> Change this
>
> if aggregate_type and aggregate_type.lower() in
> ('avg', 'max', 'min') and plot_type != 'bar':
>
> to this
>
> if aggregate_type and plot_type != 'bar':
>
> See if that works for all your graphs.
>
> -tk
>
>
> On Tue, Mar 9, 2021 at 3:21 AM Karen K <[email protected]> wrote:
>
>> I applied the patch. Then I removed all the PNG files in the HTML
>> directory, that they need to be re-created. Then I re-started WeeWX and
>> waited for the next archive interval to come. I used Seasons skin.
>>
>> There was no error message in syslog.
>>
>> I checked all the plots in all time spans for their respective unit
>> labels. All the unit labels were correct, including that of the radiation
>> energy integral.
>>
>> Please, could you include "energy_integral" in the list in line 185 of
>> imagegenerator.py together with "min", "max", and "avg" in order to put the
>> point into the middle of the aggregation interval for that aggregation
>> type, too?
>>
>> Tom Keffer schrieb am Montag, 8. März 2021 um 20:38:58 UTC+1:
>>
>>> Karen,
>>>
>>> Can you apply this simple patch to imagegenerator.py? Let me know if
>>> that fixes the problem.
>>>
>>> diff --git a/bin/weewx/imagegenerator.py b/bin/weewx/imagegenerator.py
>>> index 4daae09a..0e33e40f 100644
>>> --- a/bin/weewx/imagegenerator.py
>>> +++ b/bin/weewx/imagegenerator.py
>>> @@ -196,7 +196,7 @@ class
>>> ImageGenerator(weewx.reportengine.ReportGenerator):
>>>
>>> # Add a unit label. NB: all will get overwritten
>>> except the
>>> # last. Get the label from the configuration
>>> dictionary.
>>> - unit_label = line_options.get('y_label',
>>> weewx.units.get_label_string(self.formatter, self.converter, var_type))
>>> + unit_label = line_options.get('y_label',
>>> self.formatter.get_label_string(new_data_vec_t[1]))
>>> # Strip off any leading and trailing whitespace so
>>> it's
>>> # easy to center
>>> plot.setUnitLabel(unit_label.strip())
>>>
>>> -tk
>>>
>>> On Mon, Mar 8, 2021 at 10:20 AM Tom Keffer <[email protected]> wrote:
>>>
>>>> I've been able to confirm that this is a bug. Issue #654.
>>>> <https://github.com/weewx/weewx/issues/654>
>>>>
>>>> Thanks, Karen. I think this bug has survived all this time because
>>>> plots of aggregation types that change the unit type are rare. XTypes are
>>>> changing that.
>>>>
>>>>
>>>>
>>>> On Mon, Mar 8, 2021 at 5:52 AM Karen K <[email protected]> wrote:
>>>>
>>>>> Ah, ok, then. That is the section in skin.conf:
>>>>> [[[monthRadiationEnergy]]]
>>>>> line_gap_fraction = 0.04
>>>>> [[[[radiation]]]]
>>>>> label = "Sonnenenergie (täglich gesamt)"
>>>>> data_type = radiation
>>>>> aggregate_type = energy_integral
>>>>> aggregate_interval = 86400
>>>>>
>>>>> Tom Keffer schrieb am Montag, 8. März 2021 um 14:33:49 UTC+1:
>>>>>
>>>>>> Yes, but I was thinking more in terms of the specification for the
>>>>>> plot in skin.conf. What observation type did you say to plot? Are you
>>>>>> accidentally plotting "radiation" instead of "energy_integral?"
>>>>>>
>>>>>> -tk
>>>>>>
>>>>>> On Mon, Mar 8, 2021 at 4:30 AM Karen K <[email protected]> wrote:
>>>>>>
>>>>>>> The database query I use is that:
>>>>>>> _result = db_manager.getSql(
>>>>>>> "SELECT SUM(radiation*interval)/60.0, "
>>>>>>> "MIN(usUnits),MAX(usUnits) FROM %s "
>>>>>>> "WHERE dateTime>? AND dateTime<=?"
>>>>>>> % db_manager.table_name,timespan)
>>>>>>>
>>>>>>> --
>>>>>>>
>>>>>> You received this message because you are subscribed to the Google
>>>>>>> Groups "weewx-development" group.
>>>>>>> To unsubscribe from this group and stop receiving emails from it,
>>>>>>> send an email to [email protected].
>>>>>>>
>>>>>> To view this discussion on the web visit
>>>>>>> https://groups.google.com/d/msgid/weewx-development/ecc0844c-2061-4efb-8fd3-93a54f00936fn%40googlegroups.com
>>>>>>>
>>>>>>> <https://groups.google.com/d/msgid/weewx-development/ecc0844c-2061-4efb-8fd3-93a54f00936fn%40googlegroups.com?utm_medium=email&utm_source=footer>
>>>>>>> .
>>>>>>>
>>>>>> --
>>>>> You received this message because you are subscribed to the Google
>>>>> Groups "weewx-development" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>>> an email to [email protected].
>>>>> To view this discussion on the web visit
>>>>> https://groups.google.com/d/msgid/weewx-development/0cb6b4b7-1bab-46cc-9e6e-63ef47e28206n%40googlegroups.com
>>>>>
>>>>> <https://groups.google.com/d/msgid/weewx-development/0cb6b4b7-1bab-46cc-9e6e-63ef47e28206n%40googlegroups.com?utm_medium=email&utm_source=footer>
>>>>> .
>>>>>
>>>> --
>> You received this message because you are subscribed to the Google Groups
>> "weewx-development" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to [email protected].
>>
> To view this discussion on the web visit
>> https://groups.google.com/d/msgid/weewx-development/a1d0c101-6f55-45bc-afc2-ca09d6ca3d0en%40googlegroups.com
>>
>> <https://groups.google.com/d/msgid/weewx-development/a1d0c101-6f55-45bc-afc2-ca09d6ca3d0en%40googlegroups.com?utm_medium=email&utm_source=footer>
>> .
>>
>
--
You received this message because you are subscribed to the Google Groups
"weewx-development" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/weewx-development/60c4fae6-f705-4f47-8432-1381def9219dn%40googlegroups.com.