Re: Radio links work only in small numbers

2024-04-04 Thread Rudolf Adamkovič
Ihor Radchenko writes: > Thanks for testing! > Applied, onto main. > https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=341a01a07 > Fixed. Thanks everyone for chiming in and fixing the problem. I have been unexpectedly busy lately, hence the radio silence [pun intended]. Great

Re: Radio links work only in small numbers

2024-02-29 Thread Noboru Ota
Ihor Radchenko writes: > org-target-link-regexp does not matter in this case. For this long > regexps I introduced a new variable org-target-link-regexps that > contains a list of shorter regexps. This list is used instead of > org-target-link-regexp. Ah! Thank you. If it is valid to compare

Re: Radio links work only in small numbers

2024-02-29 Thread Ihor Radchenko
Noboru Ota writes: > - MAX_BUF_SIZE in src/regex-emacs.c is, if I understand the comment > around it correctly, 2^15 bytes = 32768 bytes. > > - `org-target-link-regexp-limit` is 2^11 bytes = 2048 bytes (?). > > - For my 5000 terms, (length org-target-link-regexp) => 103603 (what's > the unit

Re: Radio links work only in small numbers

2024-02-29 Thread Noboru Ota
Ihor Radchenko writes: > Thanks for testing! > Applied, onto main. > https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=341a01a07 > Fixed. You're very welcome. This is amazing. Thank you so much. I have rebased the source to the latest main and also tested with 10,000 dummy entries.

Re: Radio links work only in small numbers

2024-02-29 Thread Ihor Radchenko
Noboru Ota writes: >> You need Www. >> The attached version of the patch should work for 5000-terms.org. > > Thank you, Ihor. With your advice, I have managed to apply the patch. It > works for 5000 terms! Thank you. Thanks for testing! Applied, onto main.

Re: Radio links work only in small numbers

2024-02-28 Thread Noboru Ota
Ihor Radchenko writes: > You need Www. > The attached version of the patch should work for 5000-terms.org. Thank you, Ihor. With your advice, I have managed to apply the patch. It works for 5000 terms! Thank you. Some observations: - Running `org-update-radio-target-regexp` takes about 15

Re: Radio links work only in small numbers

2024-02-28 Thread Ihor Radchenko
nobiot writes: >> May you try the attached patch? > > I would love to help this patch move forward and would be happy to try > the patch, if this is not going to waste anyone's time: > > (1) I took the liberty of creating two test Org files we can use on > sr.ht:

Re: Radio links work only in small numbers

2024-02-28 Thread nobiot
Hi Ihor, Rudy, and everyone, > May you try the attached patch? I would love to help this patch move forward and would be happy to try the patch, if this is not going to waste anyone's time: (1) I took the liberty of creating two test Org files we can use on sr.ht:

Re: Radio links work only in small numbers

2024-02-19 Thread Ihor Radchenko
Rudolf Adamkovič writes: > All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured > that I may as well ask Org to make links for me. So, I replaced all the > ~4000 headings in my notes with radio <<>>. However, Org now > errors out with "Regular expression too big". > > Does

Re: Radio links work only in small numbers

2022-12-28 Thread Daniel Clemente
Hi, I also found this limitation, and my solution was to disable radio links and to replace them by a manual approach: 1. I disabled the call to (org-update-radio-target-regexp) in org.el. Well, I added a boolean org-inhibit-startup-radio-refresh, that works in a similar way to

Re: Radio links work only in small numbers

2022-12-19 Thread Timothy
Hi All, > It’s not like we cannot fix this by splitting the regexp manually into > prefix tree. > > The question is performance though. For reference, I ran into this issue with Org Glossary, and did exactly this: see .

Re: Radio links work only in small numbers

2022-12-19 Thread Rudolf Adamkovič
Max Nikulin writes: > On 14/12/2022 06:10, Rudolf Adamkovič wrote: >> >> All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured >> that I may as well ask Org to make links for me. So, I replaced all the >> ~4000 headings in my notes with radio <<>>. However, Org now >>

Re: Radio links work only in small numbers

2022-12-14 Thread Ihor Radchenko
Max Nikulin writes: > On 14/12/2022 06:10, Rudolf Adamkovič wrote: >> >> All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured >> that I may as well ask Org to make links for me. So, I replaced all the >> ~4000 headings in my notes with radio <<>>. However, Org now >>

Re: Radio links work only in small numbers

2022-12-14 Thread Max Nikulin
On 14/12/2022 06:10, Rudolf Adamkovič wrote: All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured that I may as well ask Org to make links for me. So, I replaced all the ~4000 headings in my notes with radio <<>>. However, Org now errors out with "Regular expression too

Re: Radio links work only in small numbers

2022-12-14 Thread Rudolf Adamkovič
Ihor Radchenko writes: > Can you try the attached patch? No luck: "Regular expression too big" Still too large: (length org-target-link-regexp) => 33219 :-( Rudy -- "Programming reliably -- must be an activity of an undeniably mathematical nature […] You see, mathematics is about

Re: Radio links work only in small numbers

2022-12-14 Thread Ihor Radchenko
Rudolf Adamkovič writes: > Greetings smart people! > > All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured > that I may as well ask Org to make links for me. So, I replaced all the > ~4000 headings in my notes with radio <<>>. However, Org now > errors out with "Regular

Radio links work only in small numbers

2022-12-13 Thread Rudolf Adamkovič
Greetings smart people! All [[link]]s in my notes perfectly match LEVEL-1 headings, so I figured that I may as well ask Org to make links for me. So, I replaced all the ~4000 headings in my notes with radio <<>>. However, Org now errors out with "Regular expression too big". Does anyone know