Skip to content

Commit e32394d

Browse files
committed
fixes
1 parent 9836afa commit e32394d

File tree

2 files changed

+35
-43
lines changed

2 files changed

+35
-43
lines changed

CosinorPy/cosinor.py

Lines changed: 19 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ def population_fit(df_pop, n_components = 2, period = 24, lin_comp= False, model
502502

503503

504504
#results, statistics, rhythm_params, X_test, Y_test, model = fit_me(x, y, n_components = n_components, period = period, lin_comp=lin_comp, model_type = model_type, alpha=alpha, plot = False, return_model = True)
505-
results, statistics, rhythm_params, X_test, Y_test, model = fit_me(x, y, n_components = n_components, period = period, plot = False, return_model = True, **kwargs)
505+
results, statistics, rhythm_params, X_test, _, model = fit_me(x, y, n_components = n_components, period = period, plot = False, return_model = True, **kwargs)
506506
if type(params) == int:
507507
params = results.params
508508
else:
@@ -577,7 +577,7 @@ def population_fit(df_pop, n_components = 2, period = 24, lin_comp= False, model
577577

578578

579579
if plot_on and plot_margins and model_type=='lin':
580-
sdev, lower, upper = wls_prediction_std(results, exog=X_fit_eval_params, alpha=0.05)
580+
_, lower, upper = wls_prediction_std(results, exog=X_fit_eval_params, alpha=0.05)
581581
plt.fill_between(X_test, lower, upper, color='#888888', alpha=0.1)
582582

583583
if plot_measurements:
@@ -634,7 +634,7 @@ def generate_permutations(n_pop1, n_pop2, N):
634634
"""
635635
def generate_permutations_all(pop1, pop2):
636636
n_pop1 = len(pop1)
637-
n_pop2 = len(pop2)
637+
#n_pop2 = len(pop2)
638638

639639
permutations = set()
640640

@@ -696,13 +696,10 @@ def permutation_test_population(df, pairs, period = 24, n_components = 2, lin_co
696696

697697
tests1 = list(df_pop1.test.unique())
698698
tests2 = list(df_pop2.test.unique())
699-
n_pop1 = len(tests1)
700-
n_pop2 = len(tests2)
699+
#n_pop1 = len(tests1)
700+
#n_pop2 = len(tests2)
701701

702-
#print(tests1)
703-
#print(tests2)
704-
705-
tests = np.array(tests1 + tests2)
702+
#tests = np.array(tests1 + tests2)
706703

707704
"""
708705
if not permutations:
@@ -881,7 +878,7 @@ def fit_me(X, Y, n_components = 2, period = 24, lin_comp = False, model_type = '
881878
if plot:
882879
if plot_margins:
883880
if model_type == 'lin':
884-
sdev, lower, upper = wls_prediction_std(results, exog=X_fit_test, alpha=0.05)
881+
_, lower, upper = wls_prediction_std(results, exog=X_fit_test, alpha=0.05)
885882
if color:
886883
plt.fill_between(X_test, lower, upper, color=color, alpha=0.1)
887884
else:
@@ -1019,7 +1016,7 @@ def fit_me(X, Y, n_components = 2, period = 24, lin_comp = False, model_type = '
10191016
plt.show()
10201017
if plot_residuals:
10211018
resid = results.resid
1022-
fig = sm.qqplot(resid)
1019+
sm.qqplot(resid)
10231020
plt.title(name)
10241021
if save_to:
10251022
plt.savefig(save_to+'_resid.pdf', bbox_inches='tight')
@@ -1076,7 +1073,7 @@ def plot_phases(acrs, amps, tests, period=24, colors = ("black", "red", "green",
10761073
ax.set_theta_direction(-1)
10771074
lines = []
10781075

1079-
for i, (acr, amp, test, color) in enumerate(zip(acrs, amps, tests, colors)):
1076+
for i, (acr, amp, _, color) in enumerate(zip(acrs, amps, tests, colors)):
10801077

10811078
"""
10821079
if "LDL" in test:
@@ -1336,7 +1333,7 @@ def compare_pairs(df, pairs, n_components = 3, period = 24, folder = "", prefix
13361333
save_to = ''
13371334

13381335
#pvalues, params, results = compare_pair_df_extended(df, test1, test2, n_components = n_comps, period = per, lin_comp = lin_comp, model_type = model_type, alpha=alpha, save_to = save_to, plot_measurements=plot_measurements)
1339-
pvalues, params, results = compare_pair_df_extended(df, test1, test2, n_components = n_comps, period = per, save_to = save_to, **kwargs)
1336+
pvalues, params, _ = compare_pair_df_extended(df, test1, test2, n_components = n_comps, period = per, save_to = save_to, **kwargs)
13401337

13411338
d = {}
13421339
d['test'] = test1 + ' vs. ' + test2
@@ -1370,11 +1367,6 @@ def compare_pairs(df, pairs, n_components = 3, period = 24, folder = "", prefix
13701367

13711368
#return multi.multipletests(P, method = 'fdr_bh')[1]
13721369

1373-
1374-
1375-
1376-
1377-
13781370
#def compare_pairs_best_models(df, df_best_models, pairs, lin_comp = False, model_type = 'lin', alpha = 0, folder = '', prefix = '', plot_measurements=True):
13791371
def compare_pairs_best_models(df, df_best_models, pairs, folder = "", prefix = "", **kwargs):
13801372
df_results = pd.DataFrame()
@@ -1402,7 +1394,7 @@ def compare_pairs_best_models(df, df_best_models, pairs, folder = "", prefix = "
14021394
save_to = ''
14031395

14041396
#pvalues, params, results = compare_pair_df_extended(df, test1, test2, n_components = n_components1, period = period1, n_components2 = n_components2, period2 = period2, lin_comp = lin_comp, model_type = model_type, alpha=alpha, save_to = save_to, plot_measurements=plot_measurements)
1405-
pvalues, params, results = compare_pair_df_extended(df, test1, test2, n_components = n_components1, period = period1, n_components2 = n_components2, period2 = period2, **kwargs)
1397+
pvalues, params, _ = compare_pair_df_extended(df, test1, test2, n_components = n_components1, period = period1, n_components2 = n_components2, period2 = period2, save_to = save_to, **kwargs)
14061398

14071399
d = {}
14081400
d['test'] = test1 + ' vs. ' + test2
@@ -1633,9 +1625,9 @@ def compare_pair_df_extended(df, test1, test2, n_components = 3, period = 24, n_
16331625
plt.plot(X_full, Y_fit_full2, 'r', label = test2)
16341626

16351627
if model_type == 'lin' and plot_margins:
1636-
sdev, lower, upper = wls_prediction_std(results, exog=X_fit_full[locs], alpha=0.05)
1628+
_, lower, upper = wls_prediction_std(results, exog=X_fit_full[locs], alpha=0.05)
16371629
plt.fill_between(X_full, lower, upper, color='black', alpha=0.1)
1638-
sdev, lower, upper = wls_prediction_std(results, exog=X_fit_full[~locs], alpha=0.05)
1630+
_, lower, upper = wls_prediction_std(results, exog=X_fit_full[~locs], alpha=0.05)
16391631
plt.fill_between(X_full, lower, upper, color='red', alpha=0.1)
16401632

16411633

@@ -1662,7 +1654,7 @@ def compare_pair_df_extended(df, test1, test2, n_components = 3, period = 24, n_
16621654
if plot_residuals:
16631655

16641656
resid = results.resid
1665-
fig = sm.qqplot(resid)
1657+
sm.qqplot(resid)
16661658
plt.title(test1 + ' vs. ' + test2)
16671659
save_to_resid = save_to.split(".")[0] + '_resid' + save_to.split(".")[1]
16681660
if save_to:
@@ -1852,7 +1844,7 @@ def compare_pair(X1, Y1, X2, Y2, test1 = '', test2 = '', n_components = 3, perio
18521844
if plot_residuals:
18531845

18541846
resid = results.resid
1855-
fig = sm.qqplot(resid)
1847+
sm.qqplot(resid)
18561848
plt.title(test1 + ' vs. ' + test2)
18571849
save_to_resid = save_to.split(".")[0] + '_resid' + save_to.split(".")[1]
18581850
if save_to:
@@ -2086,7 +2078,7 @@ def compare_pair_extended(X1, Y1, X2, Y2, test1 = '', test2 = '', n_components1
20862078
if plot_residuals:
20872079

20882080
resid = results.resid
2089-
fig = sm.qqplot(resid)
2081+
sm.qqplot(resid)
20902082
plt.title(test1 + ' vs. ' + test2)
20912083
save_to_resid = save_to.split(".")[0] + '_resid' + save_to.split(".")[1]
20922084
if save_to:
@@ -2519,8 +2511,8 @@ def compare_nonlinear(X1, Y1, X2, Y2, test1 = '', test2 = '', min_per = 18, max_
25192511
plt.plot(X1, Y1, 'ko', markersize=1, label = test1)
25202512
plt.plot(X2, Y2, 'ro', markersize=1, label = test2)
25212513

2522-
Y_fit1 = Y_fit[H == 0]
2523-
Y_fit2 = Y_fit[H == 1]
2514+
#Y_fit1 = Y_fit[H == 0]
2515+
#Y_fit2 = Y_fit[H == 1]
25242516

25252517
plt.plot(X_full, Y1_full, 'k', label = 'fit '+test1)
25262518
plt.plot(X_full, Y2_full, 'r', label = 'fit '+test2)
@@ -2542,7 +2534,7 @@ def compare_nonlinear(X1, Y1, X2, Y2, test1 = '', test2 = '', min_per = 18, max_
25422534
if plot_residuals:
25432535

25442536
resid = Y-Y_fit
2545-
fig = sm.qqplot(resid)
2537+
sm.qqplot(resid)
25462538
plt.title(test1 + ' vs. ' + test2)
25472539
save_to_resid = save_to + '_resid'
25482540
if save_to:

CosinorPy/cosinor1.py

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -74,12 +74,12 @@ def plot_pair(data, results, test1='1', test2='2', plot_measurements=True, save_
7474

7575
D1 = np.column_stack((H1_fit, rrr_fit_M, sss_fit_M, H1_fit*rrr_fit_M, H1_fit*sss_fit_M))
7676
D1 = sm.add_constant(D1, has_constant='add')
77-
sdev, lower, upper = wls_prediction_std(results, exog=D1, alpha=0.05)
77+
_, lower, upper = wls_prediction_std(results, exog=D1, alpha=0.05)
7878
plt.fill_between(X_fit_M, lower, upper, color=color1, alpha=0.1)
7979

8080
D2 = np.column_stack((H2_fit, rrr_fit_M, sss_fit_M, H2_fit*rrr_fit_M, H2_fit*sss_fit_M))
8181
D2 = sm.add_constant(D2, has_constant='add')
82-
sdev, lower, upper = wls_prediction_std(results, exog=D2, alpha=0.05)
82+
_, lower, upper = wls_prediction_std(results, exog=D2, alpha=0.05)
8383
plt.fill_between(X_fit_M, lower, upper, color=color2, alpha=0.1)
8484

8585

@@ -157,7 +157,7 @@ def plot_single(data, results, test='', plot_measurements=True, save_to='', plot
157157
D = sm.add_constant(D, has_constant='add')
158158

159159
if plot_margins:
160-
sdev, lower, upper = wls_prediction_std(results, exog=D, alpha=0.05)
160+
_, lower, upper = wls_prediction_std(results, exog=D, alpha=0.05)
161161
plt.fill_between(X_fit, lower, upper, color=color, alpha=0.1)
162162
#plt.fill_between(X_fit, lower, upper, color='#888888', alpha=0.1)
163163

@@ -402,7 +402,7 @@ def population_fit_cosinor(df_pop, period, save_to='', alpha = 0.05, plot_on = T
402402

403403
for test in tests:
404404
x,y = np.array(df_pop[df_pop.test == test].x), np.array(df_pop[df_pop.test == test].y)
405-
fit_results, amp, acr, statistics = fit_cosinor(x, y, period = period, save_to=save_to, plot_on = False)
405+
fit_results, amp, acr, _ = fit_cosinor(x, y, period = period, save_to=save_to, plot_on = False)
406406
if plot_on and plot_individuals:
407407
X_fit = np.linspace(min(x), max(x), 100)
408408
rrr_fit= np.cos(2*np.pi*X_fit/period)
@@ -555,27 +555,27 @@ def population_test_cosinor(pop1, pop2):
555555
mesors2 = params2[:,0]
556556
mesor2 = means2[0]
557557
betas1 = params1[:,1]
558-
beta1 = means1[1]
558+
#beta1 = means1[1]
559559
betas2 = params2[:,1]
560-
beta2 = means2[1]
560+
#beta2 = means2[1]
561561
gammas1 = params1[:,2]
562-
gamma1 = means1[2]
562+
#gamma1 = means1[2]
563563
gammas2 = params2[:,2]
564-
gamma2 = means2[2]
565-
amps1 = params1[:,3]
564+
#gamma2 = means2[2]
565+
#amps1 = params1[:,3]
566566
amp1 = means1[3]
567-
amps2 = params2[:,3]
567+
#amps2 = params2[:,3]
568568
amp2 = means2[3]
569-
acrs1 = params1[:,4]
569+
#acrs1 = params1[:,4]
570570
acr1 = means1[4]
571-
acrs2 = params2[:,4]
571+
#acrs2 = params2[:,4]
572572
acr2 = means2[4]
573573

574574
M = (k1*mesor1 + k2*mesor2)/K
575575
A = (k1*amp1 + k2*amp2)/K
576576
FI = (k1*acr1 + k2*acr2)/K
577-
BETA = (k1*beta1 + k2*beta2)/K
578-
GAMMA = (k1*gamma1 + k2*gamma2)/K
577+
#BETA = (k1*beta1 + k2*beta2)/K
578+
#GAMMA = (k1*gamma1 + k2*gamma2)/K
579579
TM = (k1 * (mesor1 - M)**2) + (k2 * (mesor2 - M)**2)
580580

581581

@@ -719,10 +719,10 @@ def test_cosinor_pairs(df, pairs, period = 24, folder = '', prefix='', plot_meas
719719
idx_group_acr = 5
720720

721721
if type(df_best_models) == int:
722-
fit_results, statistics_raw, statistics_trans, global_test_amp, ind_test_amp, global_test_acr, ind_test_acr = test_cosinor_pair(df_pair, period)
722+
fit_results, _, statistics_trans, _, ind_test_amp, _, ind_test_acr = test_cosinor_pair(df_pair, period)
723723
else:
724724
period = df_best_models[df_best_models.test == test1].period.iloc[0]
725-
fit_results, statistics_raw, statistics_trans, global_test_amp, ind_test_amp, global_test_acr, ind_test_acr = test_cosinor_pair(df_pair, period)
725+
fit_results, _, statistics_trans, _, ind_test_amp, _, ind_test_acr = test_cosinor_pair(df_pair, period)
726726
#print(period)
727727

728728
acr1, acr2 = statistics_trans['values'][idx_acr], statistics_trans['values'][idx_group_acr]

0 commit comments

Comments
 (0)