Hide traceback for epyccel and lambdify errors #1869
Conversation
|
Here is your checklist. Please tick items off when you have completed them or determined that they are not necessary for this pull request:
|
|
Hello again! Thank you for this new pull request 🤩. Please begin by requesting your checklist using the command |
![pyccel-bot[bot]](https://avatars.githubusercontent.com/in/364561?s=60&v=4){kind=small}
|
Hey @pyccel/pyccel-dev ! @EmilyBourne has just created this great new pull request! Check it out and let me know what you think! |
EmilyBourne
added
the
next release
|
Hey @yguclu, @EmilyBourne, this PR is looking pretty good. @EmilyBourne and @mustapha-belbiad think it is ready to merge. Could you add your expertise to confirm that this follows all the coding conventions and fits in Pyccel's future plans? Thanks 😄 |
pyccel-bot
bot
added
the
Ready_for_review
|
/bot run pr_tests |
pyccel-bot
bot
added
Ready_for_review
|
Hey @yguclu, @EmilyBourne, this PR is looking pretty good. @EmilyBourne and @mustapha-belbiad think it is ready to merge. Could you add your expertise to confirm that this follows all the coding conventions and fits in Pyccel's future plans? Thanks 😄 |
pyccel-bot
bot
removed
the
Ready_for_review
|
@EmilyBourne, @yguclu has a few questions/comments about your code. Can you go through and see if you agree with them. If not go ahead and explain why. Once you've adressed all the comments let me know with |
Co-authored-by: Yaman Güçlü <yaman.guclu@gmail.com>
pyccel-bot
bot
added
the
Ready_for_review
|
@yguclu, @EmilyBourne has been working hard and thinks that they have now replied to or fixed all your comments. Could you take another look at the PR and see if you can approve now? |
pyccel-bot
bot
removed
the
Ready_for_review
|
@yguclu, @EmilyBourne has been working hard and thinks that they have now replied to or fixed all your comments. Could you take another look at the PR and see if you can approve now? |
pyccel-bot
bot
added
the
Ready_for_review
pyccel-bot
bot
added
Ready_to_merge
Add a `try/except` around internal Pyccel calls to catch all `PyccelError`s. This allows the shortest possible traceback to be shown to the user when the error is recognised, handled and reported by Pyccel. Fixes #1868.
Add a `try/except` around internal Pyccel calls to catch all `PyccelError`s. This allows the shortest possible traceback to be shown to the user when the error is recognised, handled and reported by Pyccel. Fixes pyccel#1868.
Add a try/except around internal pyccel calls to catch all
PyccelErrors. This allows the shortest possible traceback to be shown to the user when the error is recognised, handled and reported by Pyccel. Fixes #1868