-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Strip/tree shake debug code #13
Comments
The gain will be less than 1Kb I think? But still, maybe there should be some way to remove that code altogether in a production buiild Just as So in fact, this project grew out of a hack and I considered it mostly abandonware 😄 Initially I didn't want to have any dependencies, but now that it is an ES6 module that must be transpiled anyway, I could as well use Typescript. But in fact, just a few days ago I was even considering a complete rewrite in C++ - so that it can be embedded in flight instruments. This year is going to be my first year with a real flight computer and I am still looking at all available options - and whatever I choose, I will surely consider an embedded port. I will probably start with an Onyx Boox. |
There is no small gain ;) Making it tree shakeable should be easy, just move the method out of the classes in a different module. You can still access the members (it might look like Those days the v8 VM is pretty efficient and I would be curious about the gain you can have with a C++ implementation. Still it is possible to compile C++ to WebASM. With regards to flight instruments I like XCTrack on Android, FlyME on Android is nice too and they can already optimize the flights in real time. I run XCTrack on a $100 android phone. |
XCTrack is currently my first choice - initially I intended to write my own 😄 - but XCTrack is so good that one cannot justify rewriting it |
I was also thinking about adding support for |
Yes there is scoring.
|
I just stumbled upon https://github.com/igclib/igclib/blob/master/src/xc/xcopt.cpp |
Also Tom Payne has a C implementation (used by the FFVL). See https://github.com/twpayne/xcscore-js/blob/master/README.md
|
It would be nice to be able to strip (or tree shake) the debug/test geojson code especially to reduce the payload for websites.
For tree shaking the debug code should be moved out of the class.
Maybe also switch to Typescript ?
The text was updated successfully, but these errors were encountered: