Skip to content
This repository was archived by the owner on Mar 11, 2022. It is now read-only.

Commit 078f7df

Browse files
authored
Support StatsProcedures as a separate package (#14)
1 parent 5af7e5b commit 078f7df

File tree

4 files changed

+13
-10
lines changed

4 files changed

+13
-10
lines changed

Project.toml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,21 @@ Reexport = "189a3867-3050-52da-a836-e630ba90ab69"
1313
StatsBase = "2913bbd2-ae8a-5f71-8c99-4fb6c76f3a91"
1414
StatsFuns = "4c63d2b9-4356-54db-8cca-17b64c39e42c"
1515
StatsModels = "3eaba693-59b7-5ba5-a881-562e759f1c8d"
16+
StatsProcedures = "d2f4e7c0-4966-47f3-9552-d63f8a71a436"
1617
Tables = "bd369af6-aec1-5ad0-b16a-f7cc5008161c"
1718
Vcov = "ec2bfdc2-55df-4fc9-b9ae-4958c2cf2486"
1819

1920
[compat]
2021
DataAPI = "1.5"
2122
DataFrames = "1"
22-
DiffinDiffsBase = "0.3.3"
23+
DiffinDiffsBase = "0.4"
2324
FixedEffectModels = "1.6"
2425
FixedEffects = "2"
2526
Reexport = "0.2, 1"
2627
StatsBase = "0.33"
2728
StatsFuns = "0.9"
2829
StatsModels = "0.6"
30+
StatsProcedures = "0.1"
2931
Tables = "1"
3032
Vcov = "0.5"
3133
julia = "1.3"

src/InteractionWeightedDIDs.jl

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,18 +10,19 @@ using Reexport
1010
using StatsBase: AbstractWeights, CovarianceEstimator, UnitWeights, PValue, TestStat, NoQuote
1111
using StatsFuns: fdistccdf
1212
using StatsModels: coefnames
13+
using StatsProcedures: _count!
1314
using Tables
1415
using Tables: getcolumn, columnnames
1516
using Vcov
1617
@reexport using DiffinDiffsBase
1718
using DiffinDiffsBase: ValidTimeType, termvars, isintercept, parse_intercept!,
18-
_count!, _groupfind, _treatnames, _parse_bycells!, _parse_subset
19+
_groupfind, _treatnames, _parse_bycells!, _parse_subset
1920

2021
import Base: show
21-
import DiffinDiffsBase: required, default, transformed, combinedargs, copyargs,
22-
valid_didargs, result, vce, treatment, nobs, outcomename, weights, treatnames,
22+
import DiffinDiffsBase: valid_didargs, vce, treatment, nobs, outcomename, weights, treatnames,
2323
dof_residual, agg, post!
2424
import FixedEffectModels: has_fe
25+
import StatsProcedures: required, default, transformed, combinedargs, copyargs, result
2526

2627
export Vcov,
2728
fe
@@ -37,7 +38,7 @@ export CheckVcov,
3738
SolveLeastSquares,
3839
EstVcov,
3940
SolveLeastSquaresWeights,
40-
41+
4142
RegressionBasedDID,
4243
Reg,
4344
RegressionBasedDIDResult,

test/procedures.jl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -551,15 +551,15 @@ end
551551
X = hcat(col0, col1)
552552
crossx = cholesky!(Symmetric(X'X))
553553
cf = crossx \ (X'y)
554-
treatcells = VecColumnTable((rel=[0, 1],))
554+
tcells = VecColumnTable((rel=[0, 1],))
555555

556556
nt = (tr=tr, solvelsweights=true, lswtnames=(), cells=cells, rows=rows,
557-
X=X, crossx=crossx, coef=cf, treatcells=treatcells, yterm=yterm, xterms=AbstractTerm[],
557+
X=X, crossx=crossx, coef=cf, treatcells=tcells, yterm=yterm, xterms=AbstractTerm[],
558558
yxterms=yxterms, yxcols=yxcols, feM=feM, fetol=1e-8, femaxiter=10000, weights=wt)
559559
ret = solveleastsquaresweights(nt...)
560560
lswt = ret.lsweights
561561
@test lswt.r === cells
562-
@test lswt.c === treatcells
562+
@test lswt.c === tcells
563563
@test lswt[lswt.r.wave_hosp.==10, 1] [-1/3, -1/3, -1/3, 1, 0] atol=fetol
564564
@test lswt[lswt.r.wave_hosp.==10, 1] [-1/3, -1/3, -1/3, 1, 0] atol=fetol
565565
@test lswt[lswt.r.wave_hosp.==10, 2] [-1/3, -1/3, -1/3, 0, 1] atol=fetol

test/runtests.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ using InteractionWeightedDIDs
33

44
using DataFrames
55
using Dates: Date, Year
6-
using DiffinDiffsBase: ValidTimeType, @fieldequal,
7-
required, default, transformed, combinedargs, _byid, valid_didargs
6+
using DiffinDiffsBase: ValidTimeType, @fieldequal, valid_didargs
87
using FixedEffectModels: Combination, nunique, _multiply
98
using FixedEffects
109
using InteractionWeightedDIDs: FETerm, _parsefeterm, getfename,
@@ -13,6 +12,7 @@ using InteractionWeightedDIDs: FETerm, _parsefeterm, getfename,
1312
solveleastsquaresweights
1413
using LinearAlgebra
1514
using StatsBase: Weights, uweights
15+
using StatsProcedures: required, default, transformed, combinedargs, _byid
1616

1717
import Base: ==
1818

0 commit comments

Comments
 (0)