-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Minutes_2023_07_04
esc edited this page Jul 5, 2023
·
1 revision
Attendees: Val, Andre, Graham, Matt, stuart FPOC (last week): Val FPOC (incoming): Stuart
NOTE: All communication is subject to the Numba Code of Conduct.
Please refer to this calendar for the next meeting date.
- NOTE: There were fewer participants than usual at the meeting (presumably) due to public holidays and so most of the discussion topics are deferred until next week.
- NEXT (Numba 0.58.0 & llvmlite 0.41.0)
- 0.58 https://github.com/numba/numba/milestone/69 28 open issues and PRs
- 0.41 https://github.com/numba/llvmlite/milestone/19 10 open issues and PRs
- Tag week: 07 Aug 2023
- PRS to discuss:
- Cache invalidation PR: https://github.com/numba/numba/pull/8396
- NumPy 1.25 PR https://github.com/numba/numba/issues/8698
- Remove conda-forge source for NumPy 1.25 packages and use anaconda.org/distro provided packages
- Updates from numba-umbrella projects.
- numba-rvsdg #50 improves testing abilities significantly
- General note - reminder that contributors need to update change logs since towncrier merge (i.e. add news snippets as per docs).
- Discourse post will be made to broadcast the change further.
- Additional work to be done to "catch up" missing news snippets since 0.57.0 (RM: action item needed)
- Summary:
- Towncrier has been added
- A GitHub workflow to add released notes to a PR
- Need to add an RST file that describes the PR
- There is a label to skip this workflow
- At the time of release the collected notes will be collated
and added to the
CHANGE_LOG
file. - Effective from this week, so you may see CI failures on new PRs
- Existing PRs will need a re-run
- Action items:
- PR template?
- in RTD docs?
- Pin an issue?
- on discourse?
- keep an eye on this for next few weeks to see how folks react
- Can we have a bot that adds a comment on the PR that
- NEP-50 and the increased prevalence of
OverflowError
handling needed. - Remove
cfunc
andcpython
wrappers from places they are not needed (e.g. internal APIs that use@jit
)- Will simplify things internally
- Make nrt compilation lazy?
- Most of this should be taken care of
- Graham will double check the "lazyness" and come back with a reproducer to highlight any remaining issues (e.g. stencil or functions with signatures trigger this)
-
Numba#8984 - Support
@guvectorize
inside@jit
- Prioritising remaining PRs from Michael Collison for future releases
- llvmlite PRs moved to backlog from 0.41.0rc1:
- #902: Add support for importing symbols from static library: Nice to have but not essential for the near term. Almost finished, so could be picked up by someone willing.
- #909: Add support for compiler-rt inclusion: Mostly just a packaging PR. Nice to have for various use cases, but not critical for the near term. May require more work than #902 to pick up depending on the level of packaging experience one has.
- Numba PRs to close:
- #8987: Add support for exceptions/asserts in CUDA device functions: Requires more work - changes the ABI, may bloat register usage and break existing linked .cu kernels, for a feature that is nice to have but ultimately can be absent (noting that exceptions in device functions have literally never worked).
- #8963: Fix math.pow with int64 argument for CUDA: Contains the suggestion to fix it in the issue, but the added tests don't test the fix, nor consider the potential new issues introduced by the fix - it would be simpler for someone to start afresh.
- Numba PRs to continue:
- #8893: Support NVRTC using the ctypes binding: Closes a gap in functionality when using the ctypes binding, for important functionality like float16 and linking external C/C++ kernels. Graham to pick up in the 0.58 timeframe.
- #8458: Testhound/cuda array reshape 7528: Reshaping arrays in CUDA will be nice to have, and better once #8893 is merged. Not so time-critical though, so Graham to pick up, but the PR is moved to the backlog for now rather than targeting 0.58.
- llvmlite PRs moved to backlog from 0.41.0rc1:
- numba#9045 - Docs should be updated for CUDA 12
- numba#9047 - Drop support for Numpy 1.21
-
numba#9049 - cpu
@guvectorize
doesn't support multiple outputs - numba#9053 - Numpy Gamma distribution not working in Numba 0.57+
- numba#9054 - numba.core.errors.NotDefinedError: Failed in nopython mode pipeline (step: analyzing bytecode) The compiler failed to analyze the bytecode. Variable '$1730setup_with.1' is not defined.
numba#9055 -np.isfortran
- numba#9052 - Numpy Gamma distribution not working in Numba 0.57+
- numba#9044 - Modify test cases to not compile-on-list
-
numba#9048 - Dynamically allocate parfor schedule.
- Stuart to ask Todd again outside of meeting
- numba#9050 - Update CODEOWNERS
- numba#9051 - Add CUDA CFG support
- llvmlite#967 - Expose library name in OrcJIT tracker
- merged - numba#9046 - copy the change log changes for 0.57.1 to main
- merged - llvmlite#968 - Update LLVM manual build instructions
- merged - llvmlite#969 - update changelog on main for v0.40.1
- Note: some PRs seem to be missing here: script at https://github.com/numba/numba/pull/9056/files needs double checking
(last numba: 9055; llvmlite 969)