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
Bug 1278685 (python-pywt) - Review Request: python-pywt - PyWavelets, wavelet transform module
Summary: Review Request: python-pywt - PyWavelets, wavelet transform module
Alias: python-pywt
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
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:
Last Closed: 2015-12-13 04:23:14 UTC
Type: ---
zbyszek: fedora-review+

Attachments (Terms of Use)

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:
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 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.

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:

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.