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

FileNotFoundError raised #58

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

venaturum
Copy link
Member

@venaturum venaturum commented Aug 1, 2024

Checklist

Description:

If logfiles are not found when parsing then a FileNotFoundError is raised

@venaturum venaturum requested a review from mattmilten August 1, 2024 12:52
Copy link
Member

@mattmilten mattmilten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@mattmilten mattmilten merged commit d3f0db9 into Gurobi:master Aug 1, 2024
6 checks passed
@ggnsjp
Copy link
Member

ggnsjp commented Sep 25, 2024

Hello, I tested main repository below command. I apologize for digging up something a little old.
this error should be handled similar way :)

python -m gurobi_logtools --write-to-dir nicenames summary.xlsx tests/assets/no-exist/*.log

=>

Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Python311\Lib\site-packages\gurobi_logtools_main
.py", line 4, in
cli(
File "C:\Python311\Lib\site-packages\gurobi_logtools\cli.py", line 29, in cli
summary = result.summary()
^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\gurobi_logtools\api.py", line 115, in summary
fill_default_parameters_nosuffix(parameters.join(summary["Version"]))
~~~~~~~^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\pandas\core\frame.py", line 4090, in getitem
indexer = self.columns.get_loc(key)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\pandas\core\indexes\range.py", line 417, in get_loc
raise KeyError(key)
KeyError: 'Version'

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.

Throw a more useful error message if no logs were parsed
3 participants