Skip to content

[df] Suggest to opt-in to Numba JIT in RDF API#22188

Open
vepadulano wants to merge 1 commit intoroot-project:masterfrom
vepadulano:rdf-numba-jit-warn
Open

[df] Suggest to opt-in to Numba JIT in RDF API#22188
vepadulano wants to merge 1 commit intoroot-project:masterfrom
vepadulano:rdf-numba-jit-warn

Conversation

@vepadulano
Copy link
Copy Markdown
Member

RDF supports calling Python functions in the Define and Filter APIs by implicit Numba-jitting the Python callable before executing it in the RDF computation graph. Given that in the future we may be able to run Python callables directly within the computation graph, and even more importantly given that numba has its limitations and it is not desirable to implicitly lock-in users to this particular technology, I propose to introduce a warning message that signals the user RDF is implicitly calling Numba JIT on one of their callables. The same functionality is still achievable opt-in.

RDF supports calling Python functions in the Define and Filter APIs by implicit Numba-jitting the Python callable before executing it in the RDF computation graph. Given that in the future we may be able to run Python callables directly within the computation graph, and even more importantly given that numba has its limitations and it is not desirable to implicitly lock-in users to this particular technology, I propose to introduce a warning message that signals the user RDF is implicitly calling Numba JIT on one of their callables. The same functionality is still achievable opt-in.
Copy link
Copy Markdown
Contributor

@siliataider siliataider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! This should ease users into the transition to pure Python callables in the future

@siliataider
Copy link
Copy Markdown
Contributor

I should also eventually update the Python section of the docs, it's good that the 2 examples explicitly specify the Numba decorator already, not the implicit syntax

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Test Results

    22 files      22 suites   3d 8h 2m 42s ⏱️
 3 847 tests  3 847 ✅ 0 💤 0 ❌
75 989 runs  75 989 ✅ 0 💤 0 ❌

Results for commit ddf105a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants