Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(Cookbook): Add Turkish translation for agent data analyst notebook #219

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Kadermiyanyedi
Copy link

Hi, This PR presents the Turkish translation of agent_data_analyst notebook and adding _toctree.yml as well.

I want to kept technical words, so I didn't translate some words such as agent.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Kadermiyanyedi Kadermiyanyedi force-pushed the add-agent-data-analyst-turkish-translation branch from 34382d8 to c60bf25 Compare October 26, 2024 17:47
@Kadermiyanyedi Kadermiyanyedi changed the title docs(Cookbok): Add Turkish translation for agent data analyst notebook docs(Cookbook): Add Turkish translation for agent data analyst notebook Oct 26, 2024
@@ -0,0 +1,267 @@
{
Copy link
Collaborator

@merveenoyan merveenoyan Oct 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ajan desek daha doğru olur, bot diyince chatbot çağrışımı yapıyor, bir de agent yerine ajan yazabiliriz kalan her yerde tutarlılık adına

Dataframelerden -> dataframe'lerden*


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ajan olarak bilerek çok çevirmek istemedim biraz tuhaf geldi açıkçası assistan olarak düşündüm ama acaba her yerde agent olarak mı bıraksak?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ajan en doğrusu bence tam altını doldurduğu için
asistanın kapsamı daha geniş

Copy link
Collaborator

@merveenoyan merveenoyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

çok teşekkürler ☺️ bir kaç minör yorum bıraktım sonra merge edebiliriz

merger'dan sonra şuraya join request göndermeyi unutmayın

@Kadermiyanyedi Kadermiyanyedi force-pushed the add-agent-data-analyst-turkish-translation branch from 0dac58b to e601a7e Compare October 28, 2024 20:16
@Kadermiyanyedi
Copy link
Author

@merveenoyan Güncel halini gönderdim, müsait oldugunda tekrar bakabilir misin? :)

@Kadermiyanyedi
Copy link
Author

@merveenoyan selam, burada yapmamı istediğin başka bir değişiklik var mı? Uygunsa PR'ı ilerletmek isterim :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants