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

Wavelet computing improvements #4

Merged
merged 2 commits into from
Jun 7, 2023
Merged

Conversation

kubo11
Copy link
Collaborator

@kubo11 kubo11 commented Jun 7, 2023

Category:

Description:

Additional information:

Affected modules and functionalities:

Key points relevant for the review:

Tests:

  • Existing tests apply
  • New tests added
    • Python tests
    • GTests
    • Benchmark
    • Other
  • N/A

Checklist

Documentation

  • Existing documentation applies
  • Documentation updated
    • Docstring
    • Doxygen
    • RST
    • Jupyter
    • Other
  • N/A

DALI team only

Requirements

  • Implements new requirements
  • Affects existing requirements
  • N/A

REQ IDs: N/A

JIRA TASK: N/A

JakubO added 2 commits June 5, 2023 16:32
This change was mainly about moving from storing wavelets as
functions to functors. Now wavelets can have extra parameters.
This introduced a challenge of making the CUDA kernel accept
these functors so templates were used.
A helper utility was also introduced on operator side. RunForName function
translates wavelet names and runs the right DALI kernel.
Discrete wavelets have been discarded since we're currently focusing
on continuous wavelet transform.

Computation of wavelet input samples has been moved to a separate cuda
kernel which should give a speedup when computing wavelets for multiple
a and b parameters.

Input wavelet samples, their scaled values and b coefficient are stored in
shared memory instead of global memory which should speedup computation.
@kubo11 kubo11 requested a review from mwdowski June 7, 2023 02:05
@kubo11 kubo11 merged commit 11df6aa into main Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant