Skip to content

Commit d4e43c5

Browse files
authored
Fix OSError: [Errno 24] Too many open files: 'nul' (#1455)
* Update model.py Added closing of save_fds. * Updated model.py for pre-commit requirements
1 parent 13aec41 commit d4e43c5

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

flaml/automl/model.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2820,7 +2820,7 @@ def __init__(self):
28202820
# Open a pair of null files
28212821
self.null_fds = [os.open(os.devnull, os.O_RDWR) for x in range(2)]
28222822
# Save the actual stdout (1) and stderr (2) file descriptors.
2823-
self.save_fds = (os.dup(1), os.dup(2))
2823+
self.save_fds = [os.dup(1), os.dup(2)]
28242824

28252825
def __enter__(self):
28262826
# Assign the null pointers to stdout and stderr.
@@ -2832,5 +2832,5 @@ def __exit__(self, *_):
28322832
os.dup2(self.save_fds[0], 1)
28332833
os.dup2(self.save_fds[1], 2)
28342834
# Close the null files
2835-
os.close(self.null_fds[0])
2836-
os.close(self.null_fds[1])
2835+
for fd in self.null_fds + self.save_fds:
2836+
os.close(fd)

0 commit comments

Comments
 (0)