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

Improve documentation memory usage warning? #2040

Closed
HadrienG2 opened this issue Mar 18, 2016 · 3 comments
Closed

Improve documentation memory usage warning? #2040

HadrienG2 opened this issue Mar 18, 2016 · 3 comments

Comments

@HadrienG2
Copy link
Contributor

As was pointed out multiple times before, compiling HPX has quite large memory requirements. Since improving these does not seem to be a priority at the moment, it is best to document them well.

Currently, the documentation has the following to say on the matter :

It is advisable that approximately 2 GB of memory per parallel process is available.

I can attest that this is quite accurate for the "make all" target, either in debug or release mode. However, today, when attempting a "make tests" on a debug build, I reached an new record 3 GB per parallel process. Unsurprisingly, the build crashed.

I think the documentation could warn about this better, but I am not sure how. Maybe the requirement should be turned into a lower bound, rather than an absolute number, i.e.

-approximately 2 GB of memory per parallel process
+at least 2 GB of memory per parallel process

What do you think?

@hkaiser
Copy link
Member

hkaiser commented Mar 18, 2016

We'd be happy to accept a PR to change this.

@HadrienG2
Copy link
Contributor Author

Here's a PR for my proposal above. Again, if someone has a better idea of how to express that requirements can vary a lot depending on what is being built, I'm all ears.

@hkaiser
Copy link
Member

hkaiser commented Mar 20, 2016

This has been fixed by merging #2042

@hkaiser hkaiser closed this as completed Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants