Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[#418] reduce memory footprint #109

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[#418] reduce memory footprint #109

wants to merge 1 commit into from

Conversation

kindly
Copy link
Contributor

@kindly kindly commented Jun 10, 2016 •
edited by Bjwebb
Loading

Move around some things to stop data being copied.
Reduce memory for when no source maps are created.

@coveralls
Copy link

coveralls commented Jun 10, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.009%) to 95.541% when pulling 82b57a1 on 418-memory-footprint into 03c36c5 on master.

@Bjwebb Bjwebb self-assigned this Jun 10, 2016
@kindly kindly force-pushed the 418-memory-footprint branch from 82b57a1 to 594eb55 Compare June 13, 2016 15:39
@coveralls
Copy link

coveralls commented Jun 13, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.01%) to 95.442% when pulling 594eb55 on 418-memory-footprint into d0f14e6 on master.

@Bjwebb Bjwebb force-pushed the 418-memory-footprint branch from 594eb55 to 8006e3f Compare June 13, 2016 15:52
@coveralls
Copy link

coveralls commented Jun 13, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.01%) to 95.442% when pulling 8006e3f on 418-memory-footprint into d0f14e6 on master.

@kindly @Bjwebb
[OpenDataServices/cove#418] Reduce memory footprint
d1fe366 
Move around some things to stop data being copied.
Reduce memory for when no source maps are created.
@Bjwebb Bjwebb force-pushed the 418-memory-footprint branch from 8006e3f to d1fe366 Compare June 24, 2016 11:00
@coveralls
Copy link

coveralls commented Jun 24, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.01%) to 95.442% when pulling d1fe366 on 418-memory-footprint into d6938b5 on master.

@Bjwebb Bjwebb removed the 5 - Review label Mar 20, 2017
@jpmckinney
Copy link
Contributor

Any reason this was not merged? Memory/time issues came up when Government Transparency Institute used flatten-tool.

@ghost ghost changed the base branch from master to main November 11, 2020 11:49
@ghost
Copy link

ghost commented Nov 26, 2020

@kindly Is this still relevant, given age - should it be closed or looked at afresh?

@BibianaC
Copy link
Member

BibianaC commented Feb 4, 2021

@kindly @Bjwebb should this PR be closed?

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

@Bjwebb Bjwebb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kindly @coveralls @jpmckinney @BibianaC @Bjwebb