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

Warning for requested/allocated thread discrepancy is printed twice. #2314

Closed
brycelelbach opened this issue Aug 29, 2016 · 3 comments · Fixed by #2317
Closed

Warning for requested/allocated thread discrepancy is printed twice. #2314

brycelelbach opened this issue Aug 29, 2016 · 3 comments · Fixed by #2317

Comments

@brycelelbach
Copy link
Member

E.g.

$ bin/future_overhead --futures=6400000 -t64
hpx::init: command line warning: --hpx:threads used when running with SLURM, requesting a larger number of threads (64) than cores have been assigned by SLURM (4), the application might not run properly.
hpx::init: command line warning: --hpx:threads used when running with SLURM, requesting a larger number of threads (64) than cores have been assigned by SLURM (4), the application might not run properly.
...

This happens because handle_arguments() gets called twice in command_line_handling.cpp, once on the initial command line (before aliases) and once after aliases, etc have been processed. It's probably somewhat annoying to fix, but it would be nice to do so.

P.S. It would also be nice to have a way to disable this flag. On some systems I can't avoid triggering this error due to odd SLURM configurations, and it makes collecting data annoying because I have to filter those warnings out of my batch script outputs.

@hkaiser
Copy link
Member

hkaiser commented Aug 29, 2016

This shouldn't be difficult to fix. We solved similar issues in the past.

P.S. It would also be nice to have a way to disable this flag. On some systems I can't avoid triggering this error due to odd SLURM configurations, and it makes collecting data annoying because I have to filter those warnings out of my batch script outputs.

We could use std::cerr instead...

@sithhell
Copy link
Member

sithhell commented Aug 29, 2016

P.S. It would also be nice to have a way to disable this flag. On some systems I can't avoid triggering this error due to odd SLURM configurations, and it makes collecting data annoying because I have to filter those warnings out of my batch script outputs.

This is already possible with the --hpx:ignore-batch-env command line switch.

@hkaiser
Copy link
Member

hkaiser commented Aug 29, 2016

This is already possible with the --hpx:ignore-batch-env command line switch.

Not sure if this is what @brycelelbach is after.

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

Successfully merging a pull request may close this issue.

3 participants