Spyder: New way to update folding regions (PR #13783) freezes Spyder

[impact27]

Issue Report Checklist

• [ ] Searched the issues page for similar reports
• [ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
• [ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
• [ ] Could not reproduce inside jupyter qtconsole (if console-related)
• [ ] Tried basic troubleshooting (if a bug/error)
  
  
  
  • [ ] Restarted Spyder
  
  
  • [ ] Reset preferences with spyder --reset
  
  
  • [ ] Reinstalled the latest version of Anaconda
  
  
  • [ ] Tried the other applicable steps from the Troubleshooting Guide
  
  
• [ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description

While using the editor, Spyder freeses for ~30s very often. I saw with the frame explorer that it was blocked in the new code intoduced by PR #13783. After reverting the merge, spyder works again.

What steps reproduce the problem?

1. 
   
   
   2.
   
3.

What is the expected output? What do you see instead?

Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

• Spyder version:
• Python version:
• Qt version:
• PyQt version:
• Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

[dalthviz]

Hi @impact27 thanks for the feedback :+1: Not sure if @andfoy or @ccordoba12 are aware of the performance issue but pinging them here just in case

[ccordoba12]

Thanks @impact27 for reporting. How long is the file that shows the freezes?

[jnsebgosselin]

I'm experiencing the same thing too. It occurs on files that are not very long, like 500 lines maybe.

[ccordoba12]

@andfoy, could you reproduce this one?

[andfoy]

It seems that textdistance is causing delays here:

[andfoy]

It seems that the lag was due to the usage of Jaro-Wrinkler distance, after replacing it with a normalized Jaccard index, the performance did improve:

[andfoy]

@impact27 @jnsebgosselin could you please check if https://github.com/spyder-ide/spyder/pull/14315 improves the editor performance?

[jnsebgosselin]

Ok, I will test it while I work. I can report back tomorrow. Thanks for working on this.

[jnsebgosselin]

I confirm that PR #14315 fixes the editor performance for me.

[ccordoba12]

Great! Thanks @jnsebgosselin for the confirmation!