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

Wrong unit for memory when submitting to SGE #40

Closed
emi80 opened this issue Jan 30, 2014 · 3 comments
Closed

Wrong unit for memory when submitting to SGE #40

emi80 opened this issue Jan 30, 2014 · 3 comments
Assignees
Labels
Milestone

Comments

@emi80
Copy link
Collaborator

emi80 commented Jan 30, 2014

Jip stores memory values in MB bytes while in SGE memory specifiers are counted in bytes if no multiplier letter follows the value.

@ghost ghost assigned emi80 Jan 30, 2014
@thasso
Copy link
Owner

thasso commented Jan 30, 2014

JIP stores memory in MB in jip.db.Job. The "if no multiplier" part scares me a bit. If the default can be configured by the SGE sys admin, this has to be a configuration option in the JIP SGE implementation so that on can adopt, or, if all SGE versions support he multipler character, maybe we can just append the character?

@emi80
Copy link
Collaborator Author

emi80 commented Jan 30, 2014

There was a typo in my previous comment.

About the multiplier I checked the SGE documentation and seems not be any configuration option for the memory unit.

I did man sge_types and I got:

memory_specifier
Memory specifiers are positive decimal, hexadecimal or octal integer constants which may be followed by a multiplier letter. Valid multiplier letters are k, K, m, M, g and G, where k means multi-ply the value by 1000, K multiply by 1024, m multiply by 1000_1000, M multiply by 1024_1024, g multiply by 1000_1000_1000 and G multiply by 1024_1024_1024. If no multiplier is present, the value is just counted in bytes.

See http://gridscheduler.sourceforge.net/htmlman/htmlman1/sge_types.html.

In this case I would just append an 'M' to the JIP stored value

@thasso
Copy link
Owner

thasso commented Jan 30, 2014

Yeah, sounds good to me.

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

No branches or pull requests

2 participants