Use a less greedy file watcher for lower dev CPU usage

[cmdcolin]

This proposes using a less greedy file watcher for our default yarn start setup

It also removes the max header size because we now use session sharing rather than huge url sizes

The less greedy file watcher helps reduce CPU usage considerably from ~6% CPU (what i see pretty much constantly while yarn start is idling, similar to the ~7% reported in the issue) to nothing (in issue it is reported as ~0.2%)

microsoft/TypeScript#31048

e.g. this is pretty much what my computer looks like 100% of the time developing jbrowse

The reason why this PR is in question is that Typescript decided not to make it their default at the time being due to some rare caveats but IMO it is rare enough that we could make it default

Laptop temp cooler===happy

[rbuels]

Nice

…

On Sun, Sep 5, 2021 at 11:41 AM Colin Diesh ***@***.***> wrote: This proposes using a less greedy file watcher for our default yarn start setup It also removes the max header size because we now use session sharing rather than huge url sizes The greedy file watcher helps reduce CPU usage considerably from ~6% CPU (what i see pretty much constantly while yarn start is idling, similar to the ~7% reported in the issue) to nothing (in issue it is reported as ~0.2%) microsoft/TypeScript#31048 <microsoft/TypeScript#31048> e.g. this is pretty much what my computer looks like 100% of the time developing jbrowse [image: Screenshot from 2021-09-05 14-06-37] <https://user-images.githubusercontent.com/6511937/132137410-821f6435-16a4-4216-9f26-a860bfbfb3be.png> The reason why this PR is in question is that Typescript decided not to make it their default at the time being due to some rare caveats but IMO it is rare enough that we could make it default Laptop temp cooler===happy ------------------------------ You can view, comment on, or merge this pull request online at: #2293 Commit Summary - Use a native file watcher File Changes - *M* products/jbrowse-desktop/package.json <https://github.com/GMOD/jbrowse-components/pull/2293/files#diff-61a30b9619765c6b4eabad9e767293c7a0a63cf77d66e68eba65ff9dedca62fe> (3) - *M* products/jbrowse-web/package.json <https://github.com/GMOD/jbrowse-components/pull/2293/files#diff-f1d261f6bb010ce285f2c2c8565c8599bf0abf6eaca8672779f80913911e85d1> (2) Patch Links: - https://github.com/GMOD/jbrowse-components/pull/2293.patch - https://github.com/GMOD/jbrowse-components/pull/2293.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2293>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAASAFNK3AWHMCDRSNTZPY3UAO2VPANCNFSM5DPCKPVA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[cmdcolin]

devs who want this can add

TSC_WATCHFILE=UseFsEventsWithFallbackDynamicPolling

to their bashrc or similar

might just close for now