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.

Sign up for GitHub

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

Jump to bottom

Feature/issue 41/interrupted time series #74

Merged
merged 65 commits into from
May 3, 2024
Merged

Feature/issue 41/interrupted time series #74

merged 65 commits into from
May 3, 2024

Conversation

davidpomerenke
Copy link
Collaborator

Implementation of impact estimation. On the way there, a lot of refactoring and adding tests. (Ideally this would be in separate branches, I know.)

Please merge #73 before merging this (it's already merged into this branch, and has additional changes on top).

davidpomerenke and others added 30 commits March 26, 2024 11:56
@davidpomerenke
Merge branch 'dev' into feature/issue-41/interrupted-time-series
cdda67f
@davidpomerenke
build(pyproject.toml): add causalpy dep and adjust pandas, numpy vers…
b2d7b15 
…ions
@davidpomerenke
feat(impact_estimators): add basic interrupted-time-series w/ regression
2eeab5a
@davidpomerenke
fix(causalpy): use version with workaround as dependency
1d87531
@davidpomerenke
feat(interrupted_time_series.py): use sklearn models
16eb1bd 
but does not work due to issue with lags
@kleinlennart @davidpomerenke
fix(climate_groups.py): add more orgs; fix JSO (UK) rename
b9ba4d2 
#27
@davidpomerenke
docs(api): add usage examples for API
e01bb82
@vogelino @davidpomerenke
feat(frontend-nextjs): Add refine scaffold
4c78148
@vogelino @davidpomerenke
feat(styles): Scaffold frontend tailwind styles; Add basic components
e26402f
@vogelino @davidpomerenke
docs(README): Update logos in README
33cbe55
@vogelino @davidpomerenke
chore(frontend): Update refine.dev's dependencies
7975ec9
@vogelino @davidpomerenke
feat(frontend): Add events table with API call to dev DB
1ccb55c
@vogelino @davidpomerenke
chore(frontend): Fix type of columns in DataTable
fea6a32
@vogelino @davidpomerenke
ci(frontend): Update node version in package.json
04c2082
@vogelino @davidpomerenke
chore(frontend): Add missing dependencies
0050420
@vogelino @davidpomerenke
chore(frontend): Add missing dependency
4b40eed
@vogelino @davidpomerenke
chore(frontend): Add missing dependency
7d0047c
@vogelino @davidpomerenke
chore(frontend): Add missing dependency
21416dc
@vogelino @davidpomerenke
chore(frontend): Add missing dependency
d304145
@vogelino @davidpomerenke
chore(frontend): Add missing dependency
ac434c9
@vogelino @davidpomerenke
ci(frontend): Attempt to fix deps
45f57d5
@vogelino @davidpomerenke
ci(frontend): Attempt to repair deployment deps
261e95f
@vogelino @davidpomerenke
refactor(Refine.dev): Get rid of refine
3559c25
@vogelino @davidpomerenke
refactor(frontend): Move events route to root
9366c8c
@davidpomerenke
feat(api): log time
f66e1c2
@davidpomerenke
refactor(api): refactor logging to newer fastapi command
7534a5f 
see https://fastapi.tiangolo.com/advanced/events/
@davidpomerenke
fix(api): remove old code, register lifespan handler
09b2e50
@davidpomerenke
fix(api): fix logging issue when api is run without uvicorn
f8ad64a
@davidpomerenke
test(mediacloud): enable mediacloud code (closes #46)
80aff8f 
The error with MediaCloud appears to have been resolved on the API side.

Closes #46
@davidpomerenke
feat(impact_estimators): add ARIMA interrupted time series estimator
07f9181
@davidpomerenke
feat(parallel.py): add parallel tqdm
1945e6a
@davidpomerenke
chore(notebooks): re-run API testing notebook
5582881 
works but different results for events vs impact endpoints
@davidpomerenke
fix(events.py): correct mediacloud query format: OR, not
c26af1a
@davidpomerenke
refactor(climate_orgs.py): rename orgs to climate_orgs
0775e82
@davidpomerenke
fix(events.py): make search by organizers case-insensitive
21a40e7
@davidpomerenke
test(events_test.py): add tests for search by organizers
1b8f14f
@davidpomerenke
feat(events.py): include aliases when filtering, and refactor
10a650d
@davidpomerenke
test(events_test.py): add tests for aliases and for wrong org names
621ee53
@davidpomerenke
fix(impact.py): fix computation of confidence interval
4e11951
@davidpomerenke
refactor(events.py, types_.py): remove individual impacts from events…
a57fd36 
… endpoint

it is still in branch individual-impacts
@davidpomerenke
refactor(notebooks): remove individual impacts from notebook
a254afc
@davidpomerenke
perf(backend-python): add tool for profiling: py-spy
5ac7498
@davidpomerenke
refactor(backend-python): harmonize datetime types
90edbaa
@davidpomerenke
test(interrupted_time_series_test.py): add tests for interrupted time…
ce4b3d3 
… series
@davidpomerenke
docs(notebooks): try normality test
5da59ea
@davidpomerenke
refactor(impact.py, types_.py): adjust to changes in interrupted_time…
d6eb040 
…_series.py
@davidpomerenke
test(data_loaders): fix some mediacloud and genios test issues
8819cd5
@davidpomerenke
Merge branch 'issue-72/google-trends' into feature/issue-41/interrupt…
6d353cc 
…ed-time-series
@davidpomerenke
refactor(pyproject.toml): remove unused dependencies
17c16db
@davidpomerenke
test(events, acled, mediacloud, api): adjust tests
9b9d438
@davidpomerenke
test(impact_test.py): add test and fix issues with impact.py
6417a28
@davidpomerenke
test(trend.py): add basic tests
1377cf0
@davidpomerenke
fix(fontend-nextjs): remove old frontend files (from bad merge)
f5aa845
@vercel Vercel
Copy link

vercel bot commented Apr 26, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
media-impact-monitor ✅ Ready (Inspect) Visit Preview May 3, 2024 8:50am

@davidpomerenke davidpomerenke mentioned this pull request Apr 26, 2024
Closed
@davidpomerenke davidpomerenke merged commit ac27efb into dev May 3, 2024
3 of 4 checks passed
@davidpomerenke davidpomerenke deleted the feature/issue-41/interrupted-time-series branch May 11, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kleinlennart kleinlennart Awaiting requested review from kleinlennart

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@davidpomerenke @kleinlennart @vogelino