Skip to content
This repository has been archived by the owner on Sep 25, 2023. It is now read-only.

Update README to reflect deprecation of cuSignal #588

Merged
merged 4 commits into from
Sep 18, 2023

Conversation

awthomp
Copy link
Member

@awthomp awthomp commented Aug 18, 2023

Deprecation Announcement for cuSignal: https://docs.rapids.ai/notices/rsn0032/

cuSignal source is moving to CuPy, and RAPIDS 23.08 is the last formal release with cuSignal.

Merge after #587

@awthomp awthomp added doc Documentation non-breaking Non-breaking change labels Aug 18, 2023
@awthomp awthomp self-assigned this Aug 18, 2023
@awthomp awthomp requested a review from jakirkham August 18, 2023 01:19
@jakirkham
Copy link
Member

Thanks Adam! 🙏


To continue your journey with GPUs, Python, and Signal Processing, please:
- [Download CuPy](https://docs.cupy.dev/en/stable/install.html)
- Visit `cupyx.scipy.signal` [source code](https://github.com/cupy/cupy/tree/main/cupyx/scipy/signal)
Copy link
Member

@leofang leofang Aug 18, 2023

Choose a reason for hiding this comment

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

Suggested change
- Visit `cupyx.scipy.signal` [source code](https://github.com/cupy/cupy/tree/main/cupyx/scipy/signal)
- Visit `cupyx.scipy.signal` [source code](https://github.com/cupy/cupy/tree/main/cupyx/scipy/signal) and [documentation](https://docs.cupy.dev/en/latest/reference/scipy_signal.html)

Copy link
Member Author

Choose a reason for hiding this comment

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

Great suggestion @leofang. I'll include this as a new bullet point in the "what's next" list.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 18, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@mnicely mnicely self-requested a review September 18, 2023 16:41
@awthomp
Copy link
Member Author

awthomp commented Sep 18, 2023

/ok to test

@awthomp
Copy link
Member Author

awthomp commented Sep 18, 2023

/merge

@rapids-bot rapids-bot bot merged commit 9cc4672 into rapidsai:branch-23.10 Sep 18, 2023
19 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc Documentation non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants