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.

Sign up for GitHub

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

Jump to bottom

Update bnet2primes.py: show error message in exception text #109

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update bnet2primes.py: show error message in exception text #109

wants to merge 3 commits into from

Conversation

kaftanski
Copy link

Makes debugging BN files much easier as you can print the exception

@hklarner
Copy link
Owner

good point. Although it is already addressed in the following commit 5ae3bc7, we can add the err message to the exception. Can you update your PR branch to the latest master to resolve conflicts?

@kaftanski
Copy link
Author

Great, thank you!

I merged the latest master into the patch branch so there are no more conflicts 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaftanski @hklarner