-
Notifications
You must be signed in to change notification settings - Fork 28
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
Very large bundle size #155
Comments
Yes, great idea! We should definitely do that. We especially draw inspiration from OpenLayers who does what you suggest. Would you like the following code? import load from 'geoblaze/load'
import identify from 'geoblaze/identify'
const georaster = await load(url);
const values = identify(georaster, point); Thank you for the suggestion! |
My quick diagnosis is that importing Additionally, Geoblaze imports the entirety of the giant |
Yes the structure like @DanielJDufour mentioned would be perfect. If the size is increased vastly by mathjs, try updating to a newer version. From 6.x on, it is as well modular and should not increase the overall size that much anymore. |
@fionawhim, awesome we didn't know about |
Also, open to PRs updating our dependencies like Chris suggested! |
Importing the package as suggested (import geoblaze from 'geoblaze';), imports everything from the library, resulting in more than 1.2mb additional scripts.
At least this is what webpack bundle analyzer is telling me.
For web projects this creates a very large footprint . Maybe you could split it into smaller modules, that can be imported separately?
The text was updated successfully, but these errors were encountered: