On Wed, Feb 12, 2020 at 1:01 PM pkx166h <[email protected]>
wrote:
>
> - *Patch*: countdown --> push
> - *Comment*:
>
> Han-wen this looks OK to push I think but can you just check David's last
> comment (in case he is advocation a change or perhaps a slpity into a new
> issue or something else that I am not getting).
>
> Otherwise please push.
> ------------------------------
>
I pushed; the discussion beyond the last LGTM does not directly refer to
the patch.
> * [issues:#5718] <https://sourceforge.net/p/testlilyissues/issues/5718/>
> Grow heap aggressively during music interpretation*
>
> *Status:* Started
> *Created:* Fri Jan 31, 2020 08:30 PM UTC by Han-Wen Nienhuys
> *Last Updated:* Mon Feb 10, 2020 07:56 PM UTC
> *Owner:* Han-Wen Nienhuys
>
> During the interpretation stage, we build up a large set of
> Grobs. Without tuning, GC is ineffective, because the Grobs are live
> data and can't be collected.
>
> To remedy this, if we see an unsuccessful GC step in the
> Score_engraver we increase the heap size explicitly
>
> Total GC overhead for mozart-hrn-3:
> without initial heap size: 1.7s
> with initial heap size set: ~1.2s
> with heap scaling: ~0.25s
>
> https://codereview.appspot.com/561390043
> ------------------------------
>
> Sent from sourceforge.net because you indicated interest in
> https://sourceforge.net/p/testlilyissues/issues/5718/
>
> To unsubscribe from further messages, please visit
> https://sourceforge.net/auth/subscriptions/
>
---
** [issues:#5718] Grow heap aggressively during music interpretation**
**Status:** Started
**Created:** Fri Jan 31, 2020 08:30 PM UTC by Han-Wen Nienhuys
**Last Updated:** Wed Feb 12, 2020 12:01 PM UTC
**Owner:** Han-Wen Nienhuys
During the interpretation stage, we build up a large set of
Grobs. Without tuning, GC is ineffective, because the Grobs are live
data and can't be collected.
To remedy this, if we see an unsuccessful GC step in the
Score_engraver we increase the heap size explicitly
Total GC overhead for mozart-hrn-3:
without initial heap size: 1.7s
with initial heap size set: ~1.2s
with heap scaling: ~0.25s
https://codereview.appspot.com/561390043
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.
_______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
