NumEconCopenhagen / projects-2024-jonas-mads

projects-2024-jonas-mads created by GitHub Classroom
MIT License
0 stars 0 forks source link

PeerFedback dataproject (jacobogmads) #5

Open ckq857 opened 5 months ago

ckq857 commented 5 months ago

Hej Jonas og Mads

Overordnet, så syntes vi I har lavet en flot, men relativ kort opgave.

  1. Det mest elegante ved jeres opgave er jeres grafer. Det er nemme at forstå og estetisk pæne.

  2. Den sværeste del af jeres kode at forstå er denne

Get data for each indicator data = {} for key, value in indicators.items(): data[key] = fred.get_series(value)

Filter data for the past 30 years for key in data: data[key] = data[key].loc['1992-01-01':'2022-01-01'] # Adjust dates based on your requirement

Det er stadig forholdsvis nemt at forstå, men man kan blive lidt i tvivl, hvad I mener med at filtrerer data. Så ville jeg nok også slette kommentarer hvor der står "your".

  1. Den overstående kode kunne blive bedre...

  2. ... ved at tilføje mere dokumentation af jeres kode.

  3. I mangler hele processen med data cleaning, hvilket i bliver bedt om at lave. Man kan haller ikke se jeres data på noget tidspunkt, så man har en fornemmelse for det.

Objectives: In your data analysis project, you should show that you can:

  1. Apply data cleaning and data structuring methods.

I kune også overveje at lave et merge med noget andet data, som vi har gennemgået hvordan vi gør i undervisningen og exercise clases.