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

RuntimeWarning: nopython is set for njit and is ignored #3

Open
fishbacp opened this issue Jan 27, 2022 · 1 comment
Open

RuntimeWarning: nopython is set for njit and is ignored #3

fishbacp opened this issue Jan 27, 2022 · 1 comment

Comments

@fishbacp
Copy link

I'm using python 3.9 on a Mac running OSX 11.6.1. I installed PyIF okay and attempted a simple example, such as the one you provided:

from PyIF import te_compute as te
import numpy as np
rand = np.random.RandomState(seed=23)
X = rand.randn(1000, 1).flatten()
Y = rand.randn(1000, 1).flatten()

TE = te.te_compute(X,Y, k=1, embedding=1, safetyCheck=True, GPU=False)
print(TE)

The resulting value was -0.0378384954814468. However, it was accompanied by a run-time warning:

Warning (from warnings module):
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/numba/core/decorators.py", line 255
    warnings.warn('nopython is set for njit and is ignored', RuntimeWarning)
RuntimeWarning: nopython is set for njit and is ignored

The result obtained using the PyInform package was 0.005256077374203942

Also: Which logarithm base is used in your calculations?

@ikegwukc
Copy link
Collaborator

ikegwukc commented Jan 28, 2022

The runtime warning will not affect the calculation of Transfer Entropy. We will do our best to fix this soon.

The example computes Transfer Entropy with random noise. Since there's no information transferred between them the result is close to 0. This is in line with the result from the comparison Python package "PyInform".

For an explanation on why the result is negative please see here: #1

We implemented the Kraskov Estimator to estimate Transfer Entropy. To see more details about this here's a link to the paper: https://arxiv.org/abs/cond-mat/0305641

This repo is currently not actively monitored. I'll do my best to reply but it is hard to make time with a full-time job

@lcdm-uiuc lcdm-uiuc locked as off-topic and limited conversation to collaborators Jan 28, 2022
@lcdm-uiuc lcdm-uiuc unlocked this conversation Jan 28, 2022
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