Make Numba optional with graceful fallback to pure Python #75#76
Open
diegoceccarelli wants to merge 4 commits intoAmenRa:masterfrom
Open
Make Numba optional with graceful fallback to pure Python #75#76diegoceccarelli wants to merge 4 commits intoAmenRa:masterfrom
diegoceccarelli wants to merge 4 commits intoAmenRa:masterfrom
Conversation
Collaborator
Author
|
@AmenRa I don't intend to merge this in |
Owner
|
@diegoceccarelli my initial experiments with eager compilation look promising. Maybe it will also alleviate the issues you are having with the compilation as well. Also, there are noticeable speed ups for metrics computation with large Qrels / Runs (~2x). Do you have any idea of the impact of your approach on the current implementation? I guess the decorators will have some impact on performance. Should we consider the option of two completely separate backends? # Use Numba backend
from ranx import Qrels, Run, evaluate
# Use Python / Numpy backend
from ranx.numpy import Qrels, Run, evaluate |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements optional Numba support with graceful fallback to pure Python execution, enabling easier debugging and reduced
dependencies while maintaining full backward compatibility.
Key Changes
Configuration System
config.py: Global configuration for enabling/disabling NumbaRANX_USE_NUMBA=falseto disable Numbaranx.set_numba_enabled(False)APIuse_numbaandset_numba_enabledin main moduleConditional Decorators (
decorators.py)@maybe_njit: Conditional njit that falls back to identity function@maybe_jit: Conditional jit decoratormaybe_prange: Falls back to regularrangewhen Numba disabledcreate_typed_dict/list: Runtime helpers for typed collectionsMetrics Refactoring
@njitto@maybe_njitnumba.typed.Listto genericlistData Structures
Benefits
For Development
For Deployment
For Users
See also #75