Skip to content

Commit

Permalink
remove priority+test
Browse files Browse the repository at this point in the history
  • Loading branch information
benclifford committed Jan 16, 2025
1 parent 5614467 commit 88dbd89
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
6 changes: 2 additions & 4 deletions parsl/executors/high_throughput/executor.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from collections import defaultdict
from concurrent.futures import Future
from dataclasses import dataclass
from typing import Callable, Dict, List, Optional, Sequence, Tuple, Union
from typing import Callable, Dict, List, Optional, Sequence, Set, Tuple, Union

import typeguard

Expand Down Expand Up @@ -357,10 +357,8 @@ def worker_logdir(self):
return self.logdir

def validate_resource_spec(self, resource_specification: dict):
"""HTEX supports the following *Optional* resource specifications:
priority: lower value is higher priority"""
if resource_specification:
acceptable_fields = {'priority'}
acceptable_fields: Set[str] = set() # add new resource spec field names here to make htex accept them
keys = set(resource_specification.keys())
invalid_keys = keys - acceptable_fields
if invalid_keys:
Expand Down
7 changes: 0 additions & 7 deletions parsl/tests/test_htex/test_resource_spec_validation.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,6 @@ def test_resource_spec_validation():
assert ret_val is None


@pytest.mark.local
def test_resource_spec_validation_one_key():
htex = HighThroughputExecutor()
ret_val = htex.validate_resource_spec({"priority": 2})
assert ret_val is None


@pytest.mark.local
def test_resource_spec_validation_bad_keys():
htex = HighThroughputExecutor()
Expand Down

0 comments on commit 88dbd89

Please sign in to comment.