Update Penalty --fa.md #315
Conversation
Create The Process of Making Structural Change in CS Internship - Cre…
Update Update Interns Profile --fa.md
Update Queue Check --fa.md
Update Technical Assistant Role and Guidelines - minor fixes
Update CS Internship Overview - Update Entrance Process
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary of Changes
Hello @Ali-Sdg90, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
This pull request significantly updates and expands the documentation for the penalty system within the CS Internship program. The changes clarify the purpose and scope of penalties, moving beyond a singular focus on integrity to encompass a wider range of program rules and commitments. The most impactful addition is a detailed breakdown of specific scenarios that incur penalties, including clear conditions that can lead to an intern's automatic exit from the program, enhancing transparency and predictability.
Highlights
- Revised Introduction and Scope: The introductory sections of the document have been rewritten to provide a broader context for penalties. The focus has shifted from solely 'integrity' to a more comprehensive framework of general program rules and commitments, aiming for a more structured and fair environment.
- Expanded Penalty Details: A significant new section titled 'شرح جزئیات پنالتیها' (Details of Penalties) has been added. This section comprehensively outlines various situations that can lead to penalties or the initiation of an intern's exit from the program.
- Categorization of Violations: The new penalty details are organized into clear categories, including 'Absence from Official Meetings,' 'Reporting and Following Up on Commitments,' 'Violation of Deadlines and Major Commitments,' and 'Interaction with Mentors and Accountability,' providing a structured overview of expected conduct.
- Defined Program Exit Conditions: Specific conditions that trigger an automatic process for an intern's exit from the program have been explicitly defined. These include not adhering to a 10-week step deadline, failing to meet personal commitments after a two-week delay, rejecting a mentor's meeting request without valid reason, or a collective decision by mentors.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments or fill out our survey to provide feedback.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Code Review
The pull request significantly improves the "Penalty" document by rewriting the introduction to be more professional and by adding a well-structured, detailed list of penalties and their consequences. The new version is much clearer and more transparent. My review includes a few suggestions to further enhance clarity and consistency, mainly by making some of the stated consequences less ambiguous and ensuring consistent formatting for key terms. Overall, this is a great update.
|
/gemeni re-review |
|
/gemini re-review |
|
It looks like you're trying to request a re-review of the pull request. The command To request a new code review for the current state of the pull request, please use the command I am unable to take actions directly, but I can provide feedback once the correct command is invoked. |
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request provides a comprehensive update to the penalty document for interns. The new version is much clearer, more professional, and better structured than the previous one. It shifts the focus from the abstract concept of 'integrity' to concrete, professional behaviors and their consequences. The penalties are well-defined and justified with real-world analogies.
I have one suggestion to improve the clarity and consistency of one of the penalty descriptions to ensure it accurately reflects the violation it pertains to.
|
Please Review @Marirood |
Ali-Sdg90
changed the base branch from
master
to
release/technical-assistant-v3.0
|
|
||
| اینتگریتی یکی از اصولی است که در برنامه **CS Internship** وجود دارد و اعضا متعهد به آن هستند. در برنامه، کانتکستها و فضاهایی برای تجربه و تمرین اینتگریتی وجود دارد تا این اصل بهصورت یک فرهنگ عملی درآید. عدم تحقق اینتگریتی از سمت اینترن، اگرچه تجربه و تمرینی است تا با نگرش به خود و فرآیند کاریاش بر آن فائق آید، اما پیامدهای خاص خود را نیز دارد. | ||
| پیامد عدم رعایت در اینتگریتی در برنامه با عنوان **پنالتی** شناخته میشود. به این معنی که هر عدم رعایت در اینتگریتی، یک پنالتی به دنبال دارد. در این مستند، لیست پنالتیهای در نظر گرفتهشده برای موقعیتهای مختلف آمده است. | ||
| برنامهٔ CS Internship، مانند هر برنامهٔ حرفهای دیگر، مجموعهای از قوانین و چارچوبهای مشخص دارد. این قوانین، پایههای اصلی نظم، هماهنگی و پیشبرد مؤثر فعالیتها در برنامه هستند و با هدف ایجاد محیطی منسجم، عادلانه و رشدمحور طراحی شدهاند. |
There was a problem hiding this comment.
برنامه CS Internship مجموعهای از سیاستها، قوانین، فرایندها و ساختارهاست.
این مجموعه
- سیاستها
- قوانین
- فرایندها
- ساختارها
بنظرم از مواردی هست بصورت یکسان در همه جا استفاده بشه.
| پیامد عدم رعایت در اینتگریتی در برنامه با عنوان **پنالتی** شناخته میشود. به این معنی که هر عدم رعایت در اینتگریتی، یک پنالتی به دنبال دارد. در این مستند، لیست پنالتیهای در نظر گرفتهشده برای موقعیتهای مختلف آمده است. | ||
| برنامهٔ CS Internship، مانند هر برنامهٔ حرفهای دیگر، مجموعهای از قوانین و چارچوبهای مشخص دارد. این قوانین، پایههای اصلی نظم، هماهنگی و پیشبرد مؤثر فعالیتها در برنامه هستند و با هدف ایجاد محیطی منسجم، عادلانه و رشدمحور طراحی شدهاند. | ||
|
|
||
| پایبندی به این چارچوبها، نهتنها نشاندهندهٔ تعهد و مسئولیتپذیری حرفهای هر اینترن است، بلکه در حفظ کیفیت تجربهٔ جمعی تمامی اعضای برنامه نیز نقش کلیدی ایفا میکند. با این حال، در مواردی که این قوانین نقض شوند یا تعهدات بهدرستی انجام نگیرند، سازوکاری تحت عنوان پنالتی در نظر گرفته شده است. |
There was a problem hiding this comment.
فکر میکنم اینجا باید به این اشاره کنیم که در محیط واقعی قوانینی وجود دارد که درصورت رعایت نکردن هر کدام از آن قوانین، تبعاتی در پی دارد.
ما در اینجا سعی داریم پیامد ناشی از رعایت نکردن قوانین رو مشخص کنیم. با این کار اینترن در زمان رعایت نکردن قوانین، با پذیرش تبعات آن شروع به هموار کردن مسیر خود به سمت رعایت تمام قوانین خواهد کرد. یعنی فرایندهایی رو برای خودش خواهد چید تا بتواند قوانین را آنطور که باید رعایت کند.
| چنانچه اینترن در جلسه فنی که برایش برنامهریزیشده است شرکت نکند و این مورد را از قبل اطلاع نداده باشد، به مدت یک هفته نمیتواند در جلسه فنی شرکت کند و این مورد در پروفایل او ثبت خواهد شد. | ||
|
|
||
|
|
||
| ### 1. عدم حضور در جلسات رسمی |
There was a problem hiding this comment.
در موارد که در قسمت شرح جزییات آمده خیلی خوب ساختار توضیح چیده شده. یعنی در قسمت اول سعی شده معنا مشخص شود و در قسمت دوم پیامد ناشی از عدم رعایت آن آمده است.
ولی دو مشکل میبینم:
مورد اولی که میبینم این هست که معنا متناسب با مورد گفته شده نیست. همچنین خیلی از جاها این معنی را نمیرساند که تو درحال یادگیری یک سری مهارت هایی هستی که در محیط کار به آنها احتیاج داری و الان با این سیستم پیامدها و پذیرش مسئولیتها در مسیر کسب این مهارتها هستی.
بنابراین فکر میکنم در مورد معنای کارها لازم هست با هم صحبت کنیم و در صورت واضح نبودنش از آقای صادقی و مهران کمک بگیریم.
همچنین جملات حاوی قضاوت از خوب و بد وجود دارد.
در کل بنظرم درست این هست که بگوییم چه کاری مطلوب هست. چه مهارتی نیاز هست.
خیلی موقع ها از نقیض آن استفاده شده. یعنی گفته شده اگر این رفتار نباشه بد هست.. این کار در فضای حرفهای بسیار منفی هست... نگران این هستم که استفاده از این روش باز ما رو به این برسونه که دیگاه فضلیت گونه به این موراد داشته باشیم و یا این دیدگاه را منتقل کنیم. اینکه اگر این قانون را رعایت نمیکنی آدم بدی هستی و اگر رعایت میکنی آدم خوبی هستی.
There was a problem hiding this comment.
هوممم... اره متوجه شدم. راجبش لازمه صحبت کنیم
No description provided.