Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-18 Thread Bastien
Ihor Radchenko writes: > Apparently 'org-marker property must always point to the beginning of > timestamp. Fixed in latest main. Confirmed, thanks a lot! -- Bastien

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-18 Thread Ihor Radchenko
Bastien writes: > with latest main, I cannot use S- (org-agenda-do-date-later) to > update the entry at point. > > Here is the backtrace I get: > > Debugger entered--Lisp error: (error "Cannot find time stamp") > error("Cannot find time stamp") Apparently 'org-marker property must always

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-18 Thread Bastien
Hi Ihor, Ihor Radchenko writes: > Bastien writes: > >> If you feel confident the change is mature enough for being merged >> into the main branch, please go ahead, this will boost the feedback. > > Merged to main just now. with latest main, I cannot use S- (org-agenda-do-date-later) to update

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-17 Thread Bastien
Ihor Radchenko writes: > Bastien writes: > >> If you feel confident the change is mature enough for being merged >> into the main branch, please go ahead, this will boost the feedback. > > Merged to main just now. Thanks! -- Bastien

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-17 Thread Ihor Radchenko
Bastien writes: > If you feel confident the change is mature enough for being merged > into the main branch, please go ahead, this will boost the feedback. Merged to main just now.

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-10-17 Thread Ihor Radchenko
Bastien writes: > If you feel confident the change is mature enough for being merged > into the main branch, please go ahead, this will boost the feedback. Merged to main just now.

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-09-25 Thread Bastien
Hi Ihor, Ihor Radchenko writes: > Timothy writes: >> Ah, I’ve already got a few patches on Org that I’m preparing for sharing on >> the >> ML, so this may make it a bit harder for me. I’ll let you know how I find it >> if >> I do give it a go though. > > My repo is up-to-date with latest

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-09-19 Thread Ihor Radchenko
Timothy writes: > Ah, I’ve already got a few patches on Org that I’m preparing for sharing on > the > ML, so this may make it a bit harder for me. I’ll let you know how I find it > if > I do give it a go though. My repo is up-to-date with latest master. You should be able to merge without too

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-09-19 Thread Timothy
Hi Ihor, Thanks for clearing up the nature of your patch-y feature request. >> p.s. I’m marking this as a patch on the tracker, since it is one :P > > This is not a real patch. I just separated changes I made in > org-element.el. Other changes include cache support across Org code and > tests.

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-09-19 Thread Ihor Radchenko
Timothy writes: > p.s. I’m marking this as a patch on the tracker, since it is one :P This is not a real patch. I just separated changes I made in org-element.el. Other changes include cache support across Org code and tests. In total it looks roughly like: 12 files changed, 2584

Re: [feature proposal] Headline caching via org-element-cache = up to 2.5x performance boost

2021-09-19 Thread Timothy
Hi Ihor, I’ve just read through your description, and this sounds very promising! Many thanks for going to the effort of tracking down this performance issue and /actually making a patch to address it/. As someone who tangles massive literate config this sounds like it could make quite a