Skip to content

Commit

Permalink
reduce memory tax for AWS resources (#795) (#796)
Browse files Browse the repository at this point in the history
Co-authored-by: Alexander Jipa <[email protected]>
  • Loading branch information
Alexander Jipa and azzhipa authored Dec 18, 2023
1 parent 9d5e3c5 commit fad46b6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions torchx/specs/named_resources_aws.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@
# ecs and ec2 have memtax and currently AWS Batch uses hard memory limits
# so we have to account for mem tax when registering these resources for AWS
# otherwise the job will be stuck in the jobqueue forever
# 97% is based on empirical observation that works well for most instance types
# 96% is based on empirical observation that works well for most instance types
# see: https://docs.aws.amazon.com/batch/latest/userguide/memory-management.html
MEM_TAX = 0.97
MEM_TAX = 0.96

# determines instance type for non-honogeneous CEs
# see https://github.com/pytorch/torchx/issues/780
Expand Down

0 comments on commit fad46b6

Please sign in to comment.