search

grisu / gricli

Grisu commandline client
7 stars 2 forks source link

simplify setting memory #70

Closed yuriyh closed 13 years ago

yuriyh commented 13 years ago

Users should be able to enter strings like "10g" , not just megabytes.

Perhaps even allow to specify memory per cpu, as in "1gpc" (1 gygabyte per cpu)

sina-masoud-ansari commented 13 years ago

If a user sets 10g with 10 cpus, is each cpu automatically given a max of 1g? I'm just wondering if a user needs to set the total memory at all...

yuriyh commented 13 years ago

changing semantics, so that memory is set per cpu maybe convenient, but this is major change, outside the scope of this issue.

sina-masoud-ansari commented 13 years ago

relating to #77 we should probably use units or GB in print global memory

ndjones commented 13 years ago

rather than changing, could we extend, so there's a flag for memory per cpu, as opposed to memory per job?

yuriyh commented 13 years ago

i.e. gricli variable memory_per_cpu = yes/no , or "10gpc" as I suggested?

yuriyh commented 13 years ago

so, can I close this one?

sina-masoud-ansari commented 13 years ago

I think so, Nick?

ndjones commented 13 years ago

Sounds like its complete. Is it in this release? If so, please shift it to the July milestone before closing.