Skip to content

Yasemin Hayırlı- Ödev Teslimi - #1

Open
yaseminhayirli wants to merge 2 commits into
murattasdemir:mainfrom
yaseminhayirli:main
Open

Yasemin Hayırlı- Ödev Teslimi#1
yaseminhayirli wants to merge 2 commits into
murattasdemir:mainfrom
yaseminhayirli:main

Conversation

@yaseminhayirli

Copy link
Copy Markdown

No description provided.

Analiz büyük oranda bitirildi, raporlama tamamlanmadı.
Analiz yapılırken Stata ve R yazılımları kullanılmıştır. Ekonometrik analizler Stata'da, grafik ise R'da yapılmıştır.
Analizin bütün aşamaları raporlanmıştır.

@murattasdemir murattasdemir left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  1. Yazdığın kodu baştan aşağı çalıştırmamışsın. Bunu yapsaydın çalışmadığını görürdün. Büyük hata 👎
  2. README.md dosyaları boş. Yeni oluşturduğun report/graphics klasöründe bir README.md dosyası yok.
  3. YH-ödev1.fdb_latekmk ve YH-ödev1.fls gibi dosyalar, TeX dosyasını derlerken ortaya çıkan yardımcı dosyalar. Bunlar paylaşılmaz. Bunların github deposuna yüklememek için .gitginore dosyasına eklersin: *.fdb* ve *.fls satırlarıyla.
  4. Tablolarda dikey çizgi kullanmayız. Makaledeki tabloda var mıydı böyle birşey?
  5. Diğer yorumlar her dosya için ayrıca yapıldı.

Comment thread 2020-fall/analysis/YH-ödev1.R Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Rstüdyo'da "projects" özelliği ile proje oluştursaydın bu satıra ihtiyacın olmayacaktı 👎
Bu satırın yer alması şunun için problem. Diğer kişilerin bunu kendi bilgisayarları için değiştirmeleri gerekiyor. Bu tür değiştirmeleri asgaride tutmamız gerekiyor.

Comment thread 2020-fall/analysis/YH-ödev1.R Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Dosya yollarını relative tutmamız, bir önceki yorumumda açıkladığım husus için önemli. R da proje mantığı bunun için önemli. Bir de, yukarıda zaten çalışma klasörünü değiştiriyorsun, burada neden absolute yol kullandığını anlayamıyorum.

Comment thread 2020-fall/analysis/YH-ödev1.R Outdated
Comment on lines 14 to 21

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Karakter kodlaması (encoding) utf8 olmadığı için Türkçe karakterler çıkmıyor benim bilgisayarımda. Bunu birçok kez söyledim. Rstudio da varsayılan kodlamayı utf8 olarak belirlemelisin. Sadece Rstudio da değil, bütün metin editörlerinde UTF8 kullanmalısın. Böylece paylaştığın kişiler senin kodlarını sorunsuz çalıştırabilir.

Comment thread 2020-fall/analysis/YH-ödev1.do Outdated
Comment on lines 15 to 17

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Bu mantıklı değil. İkinci çeyrekte doğanların kuklası neden dum2 oluyor? Değişken isimlerini seçerken, değişkenin içeriğine ilişkin maksimum bilgiyi vermesine dikkat etmelisin. Bu yanıltıcı.

Suggested change
gen dum = (QOB==1) /* Generating binary dummy variable for born in the first quarter of year */
gen dum1 = (QOB==2) /* Generating binary dummy variable for born in the second quarter of year */
gen dum2 = (QOB==3) /* Generating binary dummy variable for born in the third quarter of year */
gen QOB1 = (QOB==1) /* Generating binary dummy variable for born in the first quarter of year */

Ayrıca bizim diğer çeryrekler için kukla değişkene ihtiyacımız yok. Onları neden oluşturuyorsun? Bu kötü kodlama pratiği 👎

Comment thread 2020-fall/analysis/YH-ödev1.do Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

R dosyasına benzer bir sorun burada da var. Stata do dosyalarında önce proje klasörünü çalışma klasörü yaparsın. Sonra diğer bütün yolları göreceli (relative) olarak kullanırsız. Böylece diğer kişiler sadece çalışma klasörü yolunu değiştirmek zorunda kalırlar.

Suggested change
use "C:\Users\Bilgisayar\Documents\YH-ödev1\2020-fall\data\raw\NEW7080.dta" /* Transfer the data */
cd "C:\Users\Bilgisayar\Documents\YH-ödev1" /* Change working directory to the project directory */
use "2020-fall\data\raw\NEW7080.dta", clear /* Transfer the data */

Ayrıca, senin kullandığın şekilde çalıştırdığında, hafızada veri varsa yazılım hata verecektir. Hafızayı temizlemen gerekiyor ilk başlarken (clear all). Ya da benim önerdiğim gibi use komutunu clear seçeneği ile birlikte kullanmalısın.
Bir de kesinlikle yapılan her işlemin kaydını tutmak için bir log dosyası kullanmalısın:

log using "YH-odev1-do.log", replace

Bu veri dosyası yüklenmeden önce bir yerde olmalı.

Comment thread 2020-fall/report/YH-ödev1.tex Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Suggested change
\begin{tabular}{||c c c c||}
\begin{tabular}{l c c c}

Comment thread 2020-fall/report/YH-ödev1.tex Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Suggested change
\multicolumn{4}{|c|}{Standart hatalar parantez içinde verilmiştir. * p<0.05, ** p<0.01, *** p<0.001} \\
\multicolumn{4}{l}{Standart hatalar parantez içinde verilmiştir. * p<0.05, ** p<0.01, *** p<0.001} \\

Comment thread 2020-fall/report/YH-ödev1.tex Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LaTeX de tırnak işaretlerinin doğru çıkması için şöyle kullanmalısın:

Suggested change
"Wald Yöntemi" adını alır. Wald Yöntemi sayesinde, yerel ortalama müdahale etkisini hesaplayabiliriz.
``Wald Yöntemi'' adını alır. Wald Yöntemi sayesinde, yerel ortalama müdahale etkisini hesaplayabiliriz.

Comment thread 2020-fall/report/YH-ödev1.tex Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Dikey çizgileri kullanmıyoruz:

Suggested change
\begin{tabular}{|p{3cm}||p{2cm}|p{2cm}|p{6cm}|}
\begin{tabular}{p{3cm}p{2cm}p{2cm}p{6cm}}

Comment thread 2020-fall/report/YH-ödev1.tex Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

TIrnak işaretlerinin hatalı göründüğünü farketmemiş olamazsın:

Suggested change
\multicolumn{4}{|c|}{1920-1929 "CENSUS: 1970"} \\
\multicolumn{4}{c}{1920-1929 ``CENSUS: 1970''} \\

Aşağıdaki benzer satırları da bu şekilde değiştirmelisin.

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