[ar] cs-229-supervised-learning, cs-229-machine-learning-tips-and-tricks

[qunaieer]

[ar] Supervised Learning

[qunaieer]

I am planning to finish this file by the end of this week. I will merge it after I complete the file.

[qunaieer]

First draft is finished, I need to go back and heavily revise it. It is still not finished.

[qunaieer]

The translation of this file is complete, according to the terminologies on: https://www.nmthgiat.com/terminology/

[shervinea]

Thank you @qunaieer for all your work! Now, let's start the review process.

[zaidalyafeai]

Will work on the review. Thanks @qunaieer .

[zaidalyafeai]

Finished half of the review @qunaieer .

[qunaieer]

Thank you @zaidalyafeai
Please let me know if you finish, and I will go through the comments.

[zaidalyafeai]

@qunaieer, I am done with the review.

[qunaieer]

Thank you @zaidalyafeai for the reivew.
How can I make changes? should I make them on the file in my machine, then do pull request? or is there a way to edit them directly here? @shervinea

[zaidalyafeai]

@qunaieer you are welcome. I think it is more convenient to make them offline then make a PR. You should use the same fork that you used initially in order to append the changes to the current PR.

[qunaieer]

I have updated the file cheatsheet-supervised-learning.md according to @zaidalyafeai comments.

[shervinea]

Thank you very much @qunaieer and @zaidalyafeai for your hard work!

I see that the machine-learning-tips-and-tricks translation has also been added to this PR. @zaidalyafeai: could you please also review it? That way, we will be able to merge these two files at once.

[zaidalyafeai]

@shervinea sure, will take a stab at it asap.

[zaidalyafeai]

@qunaieer finished the review, excellent translation with minimal changes.

[qunaieer]

Thank you @zaidalyafeai for the review. I have changed the file accordingly and made a pull request.
I am not sure how to just select the changed file when doing pull request! It sends all the files I worked on previously.

[zaidalyafeai]

@qunaieer you can commit a certain file by git commit -m "my message" path/to/my/file

[shervinea]

Thank you both @qunaieer and @zaidalyafeai for all your work. On my side, I added a few commits to:

• synchronize @qunaieer's fork to the repository's latest state
• restore a template that had been overwritten by the proposed merge and move the associated translation to the [ar] directory
• add your names in the contributors file

@zaidalyafeai: would you have any more comments or would you say that this PR is now ready for merge?

[qunaieer]

Thank you @shervinea
As we discussed here #84 (comment)
It seems I need to add some tags to enable rtl. I will do this to the files and let you know.
Is this the method used for the other rtl languages (e.g., Persian)?

[shervinea]

@qunaieer Yes exactly, wrapping translations within rtl tags would be awesome (e.g. as in #129). Thank you for being so meticulous and for thinking of it!

[qunaieer]

Hello @shervinea I have updated the two files with RTL tags.

[shervinea]

Thank you very much @qunaieer, it looks great! Moving forward with the merge.