search

pttry / pttrobo

Automation of PTT data
https://pttry.github.io/pttrobo/
Other
1 stars 0 forks source link

yaml_to_excel tyhjät vuodet #4

Closed jhuovari closed 2 years ago

jhuovari commented 2 years ago

Exceleiden vuosien pitää olla tietyllä paikalla, ja aloitusvuosi määrätyy nyt star_year argumentilla. Se on tehty kuitenkin filteröinnillä, joten aloitusvuotta lyhyemmät sarjat alkavat sarjat alkuvuodesta, eli väärästä.

Puuttuvat vuodet pitäisi lisätä exeliin tyhjinä.

jhuovari commented 2 years ago

Se toimii nyt hyvin vuosihavainnilla, mutta #6 jälkeen "Vuosi" voi olla myös kuukausia/neljännesvuosia. Nopeana korjauksena voisi laittaa toimivaan vaan vuosidatalla. Kuukausidatalla oikea alkua ei niin tärkeää.

itkonen commented 2 years ago

Nyt ehkä ymmärsin väärin, mutta puuttuvat vuodet väännetään mukaan aggregoinnin jälkeen, jolloin muuttujanimi on jo "Vuosi".

jhuovari commented 2 years ago

Joo, mutta mä muutin sen niin, että jos "alkuperäinen" ja muu kun vuosidata, niin vain time nimi muutetaan Vuodeksi, ei muuteta lubridate::year. Tarvitsin datoihin myös kk data.

From: Juha Itkonen @.> Sent: Wednesday, December 22, 2021 1:52:50 PM To: pttry/pttrobo @.> Cc: Janne Huovari @.>; Author @.> Subject: Re: [pttry/pttrobo] yaml_to_excel tyhjät vuodet (Issue #4)

Nyt ehkä ymmärsin väärin, mutta puuttuvat vuodet väännetään mukaan aggregoinnin jälkeen, jolloin muuttujanimi on jo "Vuosi".

— Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpttry%2Fpttrobo%2Fissues%2F4%23issuecomment-999519422&data=04%7C01%7Cjanne.huovari%40ptt.fi%7Cb42251e36823447bdd4908d9c5419554%7C67baeb91847d43fe9b3b318d336c193d%7C0%7C0%7C637757707731257232%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=yZJnoLnwAhAn6hjyi%2B9mbRz2OCNKfy%2FRIbM%2Fl2c9WHU%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAQHTUAMNRJ2M5SV7TQYWM3USG3ZFANCNFSM5KNPATHQ&data=04%7C01%7Cjanne.huovari%40ptt.fi%7Cb42251e36823447bdd4908d9c5419554%7C67baeb91847d43fe9b3b318d336c193d%7C0%7C0%7C637757707731413481%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=FuzVS9z9S4E2n93wIfrA%2Bpr56Uy%2BA3TqDidJjWB%2BgwI%3D&reserved=0. Triage notifications on the go with GitHub Mobile for iOShttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04%7C01%7Cjanne.huovari%40ptt.fi%7Cb42251e36823447bdd4908d9c5419554%7C67baeb91847d43fe9b3b318d336c193d%7C0%7C0%7C637757707731413481%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=fDh34ifrPFlu007Mxr4A9NPdt5Nf9rmaKPqhpZFZiqI%3D&reserved=0 or Androidhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04%7C01%7Cjanne.huovari%40ptt.fi%7Cb42251e36823447bdd4908d9c5419554%7C67baeb91847d43fe9b3b318d336c193d%7C0%7C0%7C637757707731413481%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=s3eGW9nSY7kPlaisnvPJLqYwh9zu0I7WVBN4n83ddO0%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

itkonen commented 2 years ago

Jes, nyt ymmärsin!

jhuovari commented 2 years ago

Nyt tämä ei taas toimi?

ptt_update_ennustedata("Elintarvikkeet", start_year = 2012) aloittaa khi 2015 vuodesta 2015 ei 2012.

Tein funktioita noiden ennustedatojen päivitysten tekemiseen.

itkonen commented 2 years ago

Tein fixin tässä: 82d931e. Koodiin oli jäänyt bugi, ja samalla toinenkin liittyen alkuperäisiin kk-sarjoihin. Tuleeko datat nyt oikein?

jhuovari commented 2 years ago

Näyttäisi tulevan. Kiitos!