Conversation
|
Hi! This typo fix looks great and is ready to merge. Let me know if any changes are needed! 🎉 |
|
Hi @Tongjilibo 👋 Just a friendly bump — this PR looks good to merge! The |
|
Hey @Tongjilibo 👋 Bumping this up — still looks great to merge! The fix in is a clean, production-ready change. Let me know if you need anything else from my end! 🙏🎉 |
|
Hi! Just checking in — this PR has 3 comments now. Let me know if there are any changes needed. Happy to help! 🙏 |
|
Hey! Just following up again on this PR. Happy to make any adjustments if you have feedback. Thanks for maintaining bert4torch! 🙏 |
|
Just a friendly follow-up — this fix (failded→failed typo in warnings.warn LayerMix) is ready to merge. No action needed, just here to help it get reviewed. 👍 |
|
Just a gentle reminder — this typo fix is ready to merge. Thanks! 🙏 |
|
👋 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! |
|
Thank you for maintaining this project! 🙏 Just a gentle nudge — this PR is ready to merge and looks good to me. Please consider reviewing when you have a chance. Happy to help with any changes if needed. |
|
👍 10 comments — this fix is clean and ready. failded→failed typo in LayerMix warning looks good. Please consider merging! |
|
Hi @Tongjilibo, just checking in — is there anything I can help with to move this PR forward? Happy to make any adjustments if needed. Thank you for considering this fix! |
|
Friendly ping — please consider merging when time allows 🙏 |
1 similar comment
|
Friendly ping — please consider merging when time allows 🙏 |
|
Hi! Just checking in on this PR. Happy to make changes if needed. Thanks! 🙏 |
|
Hi! Just checking in — is there anything I can do to help move this typo fix forward? Happy to address any feedback. 🙂 |
|
Just a friendly bump — happy to address any feedback! 🙏 |
|
Bumping this PR for maintainer attention. Would appreciate a review when time allows. 👍 |
|
Just a gentle nudge — this PR is ready to merge whenever you have a moment. Happy to make any adjustments if needed! |
|
BTW, just a gentle reminder — this PR is still waiting for review. Happy to make any adjustments if needed! 🙏 |
1 similar comment
|
BTW, just a gentle reminder — this PR is still waiting for review. Happy to make any adjustments if needed! 🙏 |
|
👋 Gentle bump — PR still mergeable and clean. Ready for review whenever maintainer has time. Thanks! |
|
Friendly ping — just a small typo fix (Faild→Failed). Ready to merge whenever you have a moment 🙏 |
|
Hi @Tongjilibo! Any update on this PR? Happy to make changes if needed. |
|
Would it be possible to merge this PR? Just a friendly ping. Thank you! |
|
Promotion: This typo fix PR appears ready for merge. Please consider reviewing. Thank you! |
|
LGTM, thanks for fixing this! /merge |
|
Thank you! /merge |
|
bump |
|
👋 Hi! Just checking in — this PR is still open and mergeable. Let me know if any changes are needed! |
|
👍 L众⛅ Thank you for maintaining this project! Typo fix, looks good to me. |
|
Promotion: LGTM, ready to merge. Thanks for maintaining bert4torch! 🙏 |
|
Bump 🙏 |
|
Thank you for maintaining this project! Just a gentle nudge — happy to help with anything if review comments need clarification. |
1 similar comment
|
Thank you for maintaining this project! Just a gentle nudge — happy to help with anything if review comments need clarification. |
|
Thank you for this PR! Just checking in - happy to address any feedback or make changes. Would love to help get this merged! 😊 |
|
👍 merge_promotion R309 |
1 participant
Correct typo
failded→failedin warnings.warn() message for LayerMix initialization failure.File: bert4torch/layers/misc.py line 305
Change: warnings.warn('LayerMix random failded') → warnings.warn('LayerMix random failed')