Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1278685 (python-pywt) - Review Request: python-pywt - PyWavelets, wavelet transform module
Summary: Review Request: python-pywt - PyWavelets, wavelet transform module
Keywords:
Status: CLOSED ERRATA
Alias: python-pywt
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: fedora-neuro python-PyMVPA
TreeView+ depends on / blocked
 
Reported: 2015-11-06 07:24 UTC by Igor Gnatenko
Modified: 2015-12-13 04:23 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-13 04:23:14 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 644899 0 medium CLOSED Review Request: python-pywt - Python wavelet transforms module 2022-05-16 11:32:56 UTC

Internal Links: 644899

Description Igor Gnatenko 2015-11-06 07:24:34 UTC 
Spec URL: https://ignatenkobrain.fedorapeople.org/neurofedora/python-pywt.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/neurofedora/python-pywt-0.3.0-1.fc24.src.rpm
Description:
PyWavelets is a Python wavelet transforms module that can do:

* 1D and 2D Forward and Inverse Discrete Wavelet Transform (DWT and IDWT)
* 1D and 2D Stationary Wavelet Transform (Undecimated Wavelet Transform)
* 1D and 2D Wavelet Packet decomposition and reconstruction
* Computing Approximations of wavelet and scaling functions
* Over seventy built-in wavelet filters and support for custom wavelets
* Single and double precision calculations
* Results compatibility with Matlab Wavelet Toolbox
Fedora Account System Username: ignatenkobrain
Comment 1 Zbigniew Jędrzejewski-Szmek 2015-11-26 22:05:35 UTC 
Original review is 644899.

Development has obviously been picked up again, so the reasons for retirement are voided.

Why do you need separate documentation for python2 and python3? Even if there are some small difference, I don't think they are important enough to pay for the overhead of packagers and build time and disk usage and potential user confusion.

Also, the docs are >1MB, so it might be nice to split them out into a separate -doc package.

Tests in /usr/lib/python3.5/site-packages/pywt/tests/ have
#!/usr/bin/env python
Should be changed to #!/usr/bin/python3.5.

Looks good otherwise. Rpmlint complains about /usr/bin/env, but has only false positives apart from that.
Comment 2 Igor Gnatenko 2015-11-28 21:17:44 UTC 
Spec URL: https://ignatenkobrain.fedorapeople.org/neurofedora/python-pywt.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/neurofedora/python-pywt-0.3.0-2.fc24.src.rpm

Fixed all issues.
Comment 3 Zbigniew Jędrzejewski-Szmek 2015-11-28 22:02:37 UTC 
- latest version is packaged
- license is OK
- license file is present, %license is used
- latest python packaging template is used
- provides/requires are sane
- package builds and installs

All looks good. Package is APPROVED.




(BTW.: I don't think that the rule of preserving timestamps on files applies if you *modify* them. So I'd use plain 'sed -i' instead of the thing with .new and renaming, but of course doing it in the complicated way is also OK ;))
Comment 4 Fedora Update System 2015-12-02 11:07:50 UTC 
python-pywt-0.3.0-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-d61d5c30f1
Comment 5 Fedora Update System 2015-12-04 01:37:14 UTC 
python-pywt-0.3.0-2.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-pywt'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-d61d5c30f1
Comment 6 Fedora Update System 2015-12-13 04:23:11 UTC 
python-pywt-0.3.0-2.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
Note You need to log in before you can comment on or make changes to this bug.