Skip to content
This repository was archived by the owner on Jan 19, 2023. It is now read-only.

Commit 836baf3

Browse files
author
Maithra
authored
Add files via upload
fixing bug with centering
1 parent 85559b9 commit 836baf3

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

cca_core.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -321,8 +321,8 @@ def get_cca_similarity(acts1, acts2, epsilon=0., threshold=0.98,
321321
return_dict["invsqrt_yy"])
322322

323323
# compute means
324-
neuron_means1 = np.mean(acts1, axis=0, keepdims=True)
325-
neuron_means2 = np.mean(acts2, axis=0, keepdims=True)
324+
neuron_means1 = np.mean(acts1, axis=1, keepdims=True)
325+
neuron_means2 = np.mean(acts2, axis=1, keepdims=True)
326326
return_dict["neuron_means1"] = neuron_means1
327327
return_dict["neuron_means2"] = neuron_means2
328328

numpy_pca.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ def get_pca(acts, compute_dirns=False):
3636
"by datapoints")
3737

3838
# center activations
39-
means = np.mean(acts, axis=0, keepdims=True)
39+
means = np.mean(acts, axis=1, keepdims=True)
4040
cacts = acts - means
4141

4242
# compute PCA using SVD

numpy_pls.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ def get_pls_similarity(acts1, acts2):
8787
S = np.abs(S)
8888

8989
# compute means
90-
neuron_means1 = np.mean(acts1, axis=0, keepdims=True)
91-
neuron_means2 = np.mean(acts2, axis=0, keepdims=True)
90+
neuron_means1 = np.mean(acts1, axis=1, keepdims=True)
91+
neuron_means2 = np.mean(acts2, axis=1, keepdims=True)
9292

9393
# collect return values
9494
return_dict = {}

pwcca.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,14 +41,14 @@ def compute_pwcca(acts1, acts2, epsilon=0.):
4141
if np.sum(sresults["x_idxs"]) <= np.sum(sresults["y_idxs"]):
4242
dirns = np.dot(sresults["coef_x"],
4343
(acts1[sresults["x_idxs"]] - \
44-
sresults["neuron_means1"])) + sresults["neuron_means1"]
44+
sresults["neuron_means1"][sresults["x_idxs"]])) + sresults["neuron_means1"][sresults["x_idxs"]]
4545
coefs = sresults["cca_coef1"]
4646
acts = acts1
4747
idxs = sresults["x_idxs"]
4848
else:
4949
dirns = np.dot(sresults["coef_y"],
5050
(acts1[sresults["y_idxs"]] - \
51-
sresults["neuron_means2"])) + sresults["neuron_means2"]
51+
sresults["neuron_means2"][sresults["y_idxs"]])) + sresults["neuron_means2"][sresults["y_idxs"]]
5252
coefs = sresults["cca_coef2"]
5353
acts = acts2
5454
idxs = sresults["y_idxs"]

0 commit comments

Comments
 (0)