PR: Allow an adjustable time difference for modification detection (Editor)

[and1bm]

For CIFS (and perhaps other network file systems), where caching file attributes is used, expecting identical timestamps causes many false positives. This hacky patch allows for some deviation.

Description of Changes

• Wrote at least one-line docstrings (for any new functions)
• Added unit test(s) covering the changes (if testable)

• Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #21877 hopefully, we'll see next week when much more testing is planed in production.

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: and1bm

[pep8speaks]

Hello @and1bm! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-06-26 15:30:21 UTC

[ccordoba12]

Hey @and1bm, thanks for your contribution! I left a small comment for you, otherwise looks good to me.

[and1bm]

Hi @ccordoba12, I reverted the changes in the translatable string.

[ccordoba12]

Thanks! I think this is ready, so we just need your confirmation that it actually works in your systems to merge it.

[ccordoba12]

@and1bm, any news about this patch? Did it work for you?

We're planning to release 5.5.6 soon and we'd like to include it in that release.

[and1bm]

@ccordoba12 sorry, we are still investigating, it seems not to fix all (?) problems. We have actimeo=600 set for our cifs mount of the homes, perhaps it's related to that number. However, students are reporting slowly and I am waiting for more detailed reports to understand the issue and cifs access times better.

[ccordoba12]

Ok, no problem. Let us know when you have more info.