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.

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

NameError: name 'curr_mse' is not defined #54

Open
RinconFer opened this issue Oct 21, 2021 · 2 comments
Open

NameError: name 'curr_mse' is not defined #54

RinconFer opened this issue Oct 21, 2021 · 2 comments

Comments

@RinconFer
Copy link

Hi,
First, thank you for your great package. I am trying to use it on my scRNAseq datasets of brain tumors and I have find something that looks like a typo?

In the 4 STEP of the tutorial: Using MELD to characterize chordin loss-of-function
there is the following line of code:
"
results = []

with Parallel(n_jobs=36) as p:
for knn in knn_range:
# doing this outside the parallel loop because building the graph takes the longest
benchmarker.fit_graph(adata.X, knn=knn)
print(knn)
curr_results = p(delayed(simulate_pdf_calculate_likelihood)(benchmarker, seed, beta)
for seed in range(25) for beta in beta_range)
curr_results = pd.DataFrame(curr_results, columns = ['MSE', 'seed', 'beta', 'knn'])
results.append(curr_mse)

results = pd.concat(results, axis=0)
"
The function call for a curr_mse variable that is not defined, nor used, anywhere else, and it causes the error in the title of this issue. It is possible that it could be curr_results or I am missing something.

I am quite new in python so if I am just making no sense please let me know.
Thank you very much
David

@dburkhardt
Copy link
Member

Ahh good catch. It should call results.append(curr_results)

@RinconFer
Copy link
Author

Great! thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants