Skip to content

Conversation

@nielsvanvelzen
Copy link

I started using Nightscout a few weeks ago but when I tried to update to the latest version a few days ago it didn't work because my server (that was compiling it) only had 512MB memory.

After some digging I found out that this was caused by the webpack config not excluding node_modules from the ruels for image + css loading.

When testing on my Windows machine memory usage went from a peak of 1.5GB to 200MB. Compile times got speed up too.

After testing I can confirm that everything is still working (tested the admin & clocks in particular)

@PieterGit
Copy link
Contributor

@nielsvanvelzen thanks for finding this bug. Please target PR's at the dev branch. I will test this PR later or perhaps @sulkaharo beats me.

@nielsvanvelzen nielsvanvelzen changed the base branch from master to dev August 10, 2018 10:47
PieterGit added a commit to PieterGit/cgm-remote-monitor that referenced this pull request Sep 29, 2018
@PieterGit
Copy link
Contributor

Reimplemented with #3953 (without reformatting). This PR signficantly reduces memory usage and built time:

  Δt 109764ms (436 modules hidden) [without this PR]
  Δt 22199ms (436 modules hidden) [with this PR]

Closing this PR. We'll merge it with #3953

@PieterGit PieterGit closed this Sep 29, 2018
@PieterGit PieterGit added this to the 0.11.0 milestone Sep 29, 2018
@PieterGit
Copy link
Contributor

@nielsvanvelzen thanks for finding and PR'ing this bug. Sorry it took so long to take action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants