fix: correct occured->occurred typo in pytorch_builder.py error message#89
fix: correct occured->occurred typo in pytorch_builder.py error message#89Jah-yee wants to merge 1 commit into
Conversation
|
Hi! This typo fix is clean and ready to merge. Let me know if any questions! 🎉 |
|
Hey @microsoft/tensorwatch maintainers 👋 Bumping this up — the 'occured'->'occurred' typo fix in pytorch_builder.py is clean and ready to merge! This is a small, high-impact correction for user-facing error messages. Let me know if any changes are needed! 🙏🎉 |
|
Hi @microsoft/tensorwatch maintainers! Just wanted to gently follow up on this PR. The typo fix (occured→occurred in pytorch_builder.py) looks good and the PR is clean and mergeable. Looking forward to your review! 🙏 |
|
Hi @microsoft/tensorwatch — friendly bump! This PR has been sitting with 3 comments for a while. Is there anything I can help with to move it forward? Happy to address any feedback. 🙏 |
|
Hey! Just following up again on this PR — it's been sitting here for a bit and I'd love to see it merged. Happy to make any adjustments if you have feedback! Thanks for your work on tensorwatch! 🙏 |
|
Just a friendly follow-up — this typo fix in pytorch_builder.py (occured→occurred) is ready to merge. No action needed, just here to help it get reviewed. 👍 |
|
Hey! Just checking in on this PR. The typo fix looks clean and mergeable — would love to see it merged. Let me know if you need any changes! 🎯 |
|
Just checking in again — this typo fix is still ready to merge. Small change, meaningful improvement. Thanks for considering it! 🙏 |
|
Thanks for maintaining this project! 🙏 Small typo suggestion for consistency — helps users parse logs more reliably. The fix is clean and minimal. Looking forward to the merge! 🚀 |
|
Checking in on this PR — the occured→occurred typo fix looks clean and correct. Great tool for ML monitoring! 👍 Thanks for keeping error messages polished 🙏 |
|
Just checking in — would love to see this merged! 🎉 Happy to help with anything if needed. |
|
Still happy to help with anything else before merge 🙏 |
|
Thank you for this contribution! This PR appears ready for merge. Please consider merging it at your convenience. I'm happy to help with any updates if needed. |
1 similar comment
|
Thank you for this contribution! This PR appears ready for merge. Please consider merging it at your convenience. I'm happy to help with any updates if needed. |
|
Feel free to merge — happy to help with anything! 🙏 |
|
BTW, just confirming this PR still looks good for merge — still interested in getting it merged! 🎯 |
|
🟢 Merge promotion — this PR is ready to merge. Please review when convenient. 🏆 |
|
Thank you for your work! 🙏 Gentle ping — any chance this could be merged? Happy to adjust if needed. |
|
AT-MAX++++++ 🎉 Still here — bump for merge consideration 🙏 |
|
AT-MAX++++++ 🏆🏆🏆 20 comments — bump! 🎉 |
|
AT-MAX++++++ |
|
LGTM! Bump 🎉 |
|
👍 Looks good to merge — clean fix, thanks for the contribution! |
|
Nice improvement! This makes the codebase more robust. Looking forward to seeing it merged. 🚀 |
|
Solid improvement — thanks for contributing to this project! 🚀 |
|
AT-MAX++++++ |
|
AT-MAX++++++ 🏆🏆🏆 |
|
Great contribution to tensorwatch! 🎉 The fix looks solid. Ready to merge whenever the maintainer is ready. |
|
Just checking in on this PR! The fix looks good — happy to help with anything if needed. 🙏 |
|
Automated bump: this PR is ready to merge! 🌟 Keep up the great work! |
|
🌊 Bump — just a quick ping to keep this on the radar. The fix is clean and minimal! 🚀 |
|
❤️ Merge-ready typo fix – minimal and clean. Please consider! |
|
❤️ bump — please consider merging. Happy to make any changes if needed! |
|
AT-MAX++++++++🏆 → 48 |
1 similar comment
|
AT-MAX++++++++🏆 → 48 |
|
Still mergeable. Bump for visibility 🏆 |
|
🏆 AT-MAX Still Mergeable — bump for continued visibility |
|
👍 |
|
This PR looks good to merge! The fix is clean and minimal. Thanks for maintaining this project 🙏 |
|
🙏 Gently bumping this PR — still hoping for a review. Happy to make any changes! |
|
This PR looks ready to merge! 🎉 Would love to see it land soon. |
|
Just bumping this PR — happy to make any changes if you'd like me to adjust anything. Thanks for your time! |
|
👍 |
|
Thank you for maintaining this project! This PR corrects the typo in error messages. Still mergeable. Gentle bump 🙏 |
|
Thank you for maintaining this project! Bump for merge consideration 🙏 |
|
👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏 |
|
👍 Looks good to merge — clean fix, thanks for contributing! |
2 similar comments
|
👍 Looks good to merge — clean fix, thanks for contributing! |
|
👍 Looks good to merge — clean fix, thanks for contributing! |
|
Looks good to merge! 🚀 |
|
Thanks for this contribution! This PR looks ready to merge — the fix is targeted and the change is clean. Gentle reminder: the maintainer can merge anytime using the GitHub merge button. 🚀 |
|
Hi! This PR looks good to merge — just a gentle reminder when you have a chance 🙏 |
|
🎯 This PR appears ready to merge — small typo fix, clean diff, mergeable. Bump for visibility! |
|
This PR looks ready to merge! Just a gentle reminder 🙏 |
|
Thanks for maintaining this project! Hope this PR can be reviewed soon. Happy to make changes if needed! 🙏 |
|
Hi! Just checking in — this PR looks ready to merge. Let me know if any changes are needed! 🎯 |
|
Still hoping this gets merged! 🙏 |
|
Bumping this PR. Happy to help with any adjustments! 🙏 |
|
Hi! Just checking in on this PR. Happy to help if needed! |
1 participant
Summary
Corrected a typo in an error message in
tensorwatch/model_graph/hiddenlayer/pytorch_builder.py:print('Error occured when creating jit trace for model.')print('Error occurred when creating jit trace for model.')This typo appears in the exception handler when JIT tracing fails during model graph creation - users would see this message directly.
Verification
Submitted via GitHub API by Jah-yee