Open Nicholas-7 opened 3 years ago
Thanks for the issue, @Nicholas-7.
I have no plans to add support to external dependencies to cusignal. This includes the packages rtlsdr
, soapysdr
, and the .sigmf
file used to test I/O. These notebooks are meant to be a learning tool for the developer on how to get started with using cusignal "in the wild".
I'm okay if we want to disable these notebooks from running CI/CD or other creative solutions.
Thanks for bringing up the issue with use_numba
. I've filed a PR to fix that.
Finally, for the cwt
error, we recently filed a fix that should make it into the next release of cusignal that addresses this error.
This issue has been labeled inactive-30d
due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d
if there is no activity in the next 60 days.
@awthomp may i propose a possible resolution?
conda install -y -c conda-forge soapysdr rtl-sdr
to the affected notebooks and PR it. That should close this up nicely without interfering too much. the issue we're trying to fix is not a CI/CD issue as much as a "I love cusignal and can't run these notebooks to see its capability in these areas, so I can imagine others" issue. SDR is something I want to get into soon and am in process of getting the certs :)
This issue has been labeled inactive-30d
due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d
if there is no activity in the next 60 days.
This issue has been labeled inactive-90d
due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.
Describe the bug Discovered errors in a few notebooks relating to the cuSignal library after running them in the 21.10 stable release. The errors were tested and found in both CentOS and Ubuntu operating systems. Not sure if these errors are a result of possible updates to the codebase or if it was an uncaught bug.
Steps/Code to reproduce bug Steps to reproduce the behavior: 1.Go to RAPIDS Sample Notebooks and clone the 21.10 branch 2.Click on the cuSignal folder 3.Run all the cells of the notebooks to produce the examples illustrated below
Expected behavior There will be several examples that will create an error. Many examples miss details that could aide in implementation. The code may be a few commits behind from the 21.10 repo.
Environment details (please complete the following information):
Additional context Examples of Discrepancies:
Example # 1 sdr_integration
Where after discovering the above I tried to remedy by reinstalling RtlSdr using the uncommented code.
Error thrown below:
Example # 2 sdr_wfm_demod
Where after discovering the above I tried to remedy by reinstalling SoapySDR using the uncommented code.
Error thrown below:
Example # 3 convolution_examples
Where use_numba=True isnt recognized as a parameter for convolve2d()
Error thrown below:
A second instance of the similar issue filtering_examples
Where use_numba=True isnt recognized as a parameter for resample_poly()
Error thrown below:
Example # 4 io_examples
Where /data/oracle/KRI-16Devices-RawData/2ft/WiFi_air_X310_3123D7B_2ft_run1.sigmf-meta is not available
Error thrown below:
Example # 5 wavelets_examples
Where we seem at one time to be expecting an integer.
Error thrown below:
Desired outcome cuSignal notebooks should be immediately ready to be replicated and implemented with less effort. Notebooks should be updated to reflect the commits made to the repositories during each release cycle. cuSignal functions and models work as expected.
Request impacts Our cuSignal notebooks are public and require accurate information - Medium Priority
@taureandyernv @fondaing @awthomp for awareness