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.

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

iedorep: Memory management issues #274

Closed
bbdaniels opened this issue Jun 7, 2022 · 2 comments · Fixed by #261
Closed

iedorep: Memory management issues #274

bbdaniels opened this issue Jun 7, 2022 · 2 comments · Fixed by #261
Assignees
Labels
major bug Command may lead to incorrect analysis resolved but not yet published Issue is fixed, but not yet published on SSC

Comments

@bbdaniels
Copy link
Contributor

Crashes when user manually sets memory in dofile -- assuming this is because Stata uses clear all or equivalent in background when memory settings are changed. Need to check.

@kbjarkefur -- Having a different issue where it seems that making so many temp copies of a 40MB dataset is causing Stata to be unable to restore or store locals appropriately. Have you seen anything like this before? It is 30 vars, 250k obs so it's not enormous, and even 100 copies of it would only make 4GB which should not be a strain on my system at all. And yet I have locals disappearing and getting the message unable to restore data due to insufficient memory in cases.

@bbdaniels bbdaniels added the major bug Command may lead to incorrect analysis label Jun 7, 2022
@bbdaniels bbdaniels linked a pull request Jun 7, 2022 that will close this issue
21 tasks
@bbdaniels
Copy link
Contributor Author

bbdaniels commented Jun 7, 2022

One option here is to not save every data state to use cf later, and instead use the hashes or signatures. This would lead to some loss of precision.... but perhaps it is worth it, at least optionally, if this is hard to solve.

@bbdaniels
Copy link
Contributor Author

Solution proposed in 975f66c

@bbdaniels bbdaniels added the resolved but not yet published Issue is fixed, but not yet published on SSC label Dec 13, 2022
@bbdaniels bbdaniels changed the title [iedorep]: Memory management issues iedorep: Memory management issues Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major bug Command may lead to incorrect analysis resolved but not yet published Issue is fixed, but not yet published on SSC
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants