search

spectriclabs / sigplot

:chart_with_upwards_trend: SigPlot provides fast interactive web-plotting for software defined radio.
Apache License 2.0
8 stars 3 forks source link

Replace lru with lru-cache #5

Open mrecachinas opened 3 years ago

mrecachinas commented 3 years ago

They have an identical API (for our purposes), but lru-cache is fully browser-ready, whereas lru has some node-specific requirements (i.e., namely the events library). In my update-tooling effort, lru caused esbuild to error, while lru-cache did not. Also, lru was last updated 5 years ago, whereas lru-cache was last updated 10 months ago.

Note: I have verified SDS functionality remained the same.

mrecachinas commented 3 years ago

Ha, the funny thing is back in May 2020, @gfaiks pinged me with a Babel/ES6 issue that was resolved by switching from lru-cache to lru...

I'm still not seeing the issue, but it's possible I'm using too new of a browser?