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

Fix sys.excepthook_org handling #149

Merged
merged 1 commit into from
Sep 18, 2023
Merged

Conversation

albertz
Copy link
Member

@albertz albertz commented Sep 18, 2023

It might not be set, depending on where the exception was raised, and depending on the entry point of Sisyphus.

Via:

Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/sisyphus-1.0.0-py3.8.egg/sisyphus/loader.py", line 54, in load_config_file
    config = importlib.import_module(module_name)
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 10[14](https://github.com/rwth-i6/i6_experiments/actions/runs/6219601859/job/16877982502?pr=175#step:8:15), in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/runner/work/i6_experiments/i6_experiments/config/common/baselines/librispeech.py", line 1, in <module>
    from i6_experiments.common.baselines.librispeech.ls100.gmm.baseline_config import run_librispeech_100_common_baseline
  File "/home/runner/work/i6_experiments/i6_experiments/recipe/i6_experiments/common/baselines/librispeech/ls100/gmm/baseline_config.py", line 6, in <module>
    from i6_experiments.common.setups.rasr import gmm_system
  File "/home/runner/work/i6_experiments/i6_experiments/recipe/i6_experiments/common/setups/rasr/__init__.py", line 1, in <module>
    from .gmm_system import *
  File "/home/runner/work/i6_experiments/i6_experiments/recipe/i6_experiments/common/setups/rasr/gmm_system.py", line 1188
    train_corpus_key=trn_c,
    ^
SyntaxError: keyword argument repeated

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "check_jobs.py", line 10, in <module>
    config_manager.load_configs([function])
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/sisyphus-1.0.0-py3.8.egg/sisyphus/loader.py", line 1[17](https://github.com/rwth-i6/i6_experiments/actions/runs/6219601859/job/16877982502?pr=175#step:8:18), in load_configs
    self.load_config_file(filename)
  File "/opt/hostedtoolcache/Python/3.8.[18](https://github.com/rwth-i6/i6_experiments/actions/runs/6219601859/job/16877982502?pr=175#step:8:19)/x64/lib/python3.8/site-packages/sisyphus-1.0.0-py3.8.egg/sisyphus/loader.py", line 59, in load_config_file
    sys.excepthook = sys.excepthook_org
AttributeError: module 'sys' has no attribute 'excepthook_org'

It might not be set, depending on where the exception was raised.
@albertz albertz merged commit adadf2c into master Sep 18, 2023
3 checks passed
@albertz albertz deleted the albert-fix-excepthook-org branch September 18, 2023 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants