Bug 1449328 - Review Request: zfp - Library for compressed numerical arrays with high throughput R/W random access
Summary: Review Request: zfp - Library for compressed numerical arrays with high throu...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
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:
TreeView+ depends on / blocked
 
Reported: 2017-05-09 16:04 UTC by Orion Poplawski
Modified: 2019-02-25 15:12 UTC (History)
2 users (show)

Fixed In Version: zfp-0.5.4-1.fc31
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-25 15:12:52 UTC
Type: Bug
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)

Description Orion Poplawski 2017-05-09 16:04:40 UTC
Spec URL: https://www.cora.nwra.com/~orion/fedora/zfp.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/zfp-0.5.1-1.el7.src.rpm
Description:
This is zfp, an open source C/C++ library for compressed numerical arrays
that support high throughput read and write random access. zfp was written by
Peter Lindstrom at Lawrence Livermore National Laboratory, and is loosely
based on the algorithm described in the following paper:

Peter Lindstrom
"Fixed-Rate Compressed Floating-Point Arrays"
IEEE Transactions on Visualization and Computer Graphics,
  20(12):2674-2683, December 2014
doi:10.1109/TVCG.2014.2346458

zfp was originally designed for floating-point data only, but has been
extended to also support integer data, and could for instance be used to
compress images and quantized volumetric data. To achieve high compression
ratios, zfp uses lossy but optionally error-bounded compression. Although
bit-for-bit lossless compression of floating-point data is not always
possible, zfp is usually accurate to within machine epsilon in near-lossless
mode.

zfp works best for 2D and 3D arrays that exhibit spatial coherence, such as
smooth fields from physics simulations, images, regularly sampled terrain
surfaces, etc. Although zfp also provides a 1D array class that can be used
for 1D signals such as audio, or even unstructured floating-point streams,
the compression scheme has not been well optimized for this use case, and
rate and quality may not be competitive with floating-point compressors
designed specifically for 1D streams.

Fedora Account System Username: orion

https://koji.fedoraproject.org/koji/taskinfo?taskID=19475761

Comment 1 Zbigniew Jędrzejewski-Szmek 2017-07-15 03:53:38 UTC
Those files have very generic names:
/usr/include/bitstream.h
/usr/include/cache.h
/usr/include/memory.h
In fact, glibc has memory.h, so this package would break stuff.
Dunno, ideally upstream would move all header files to /usr/include/zfp/ (except for /usr/include/zfp.h).

There are tests, you could run them in %check.

Comment 2 Orion Poplawski 2019-02-24 18:41:30 UTC
Looks like upstream cleaned up cache.h and memory.h, though bitstream.h remains.  I've filed https://github.com/LLNL/zfp/issues/49  

Spec URL: https://www.cora.nwra.com/~orion/fedora/zfp.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/zfp-0.5.4-1.el7.src.rpm

* Sun Feb 24 2019 Orion Poplawski <orion> - 0.5.4-1
- Update to 0.5.4

Comment 3 Zbigniew Jędrzejewski-Szmek 2019-02-24 20:46:22 UTC
BR: gcc and BR: gcc-c++ are missing.

/usr/include/bitstream.h is a bit unfortunate, but it doesn't conflict with anything, so it's acceptable.

+ package name is OK
+ license is specified correctly (BSD)
+ license is acceptable for fedora
+ latest version
+ builds OK
+ Provides/Requires look OK
+ rpmlint finds no issues (just false positives about spelling and outdated demands for ldconfig)

Package is APPROVED.

Comment 4 Orion Poplawski 2019-02-25 00:04:02 UTC
Thanks for the review.  Upstream is aware of the issue an is looking to rework the names.

Comment 5 Gwyn Ciesla 2019-02-25 14:37:14 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/zfp


Note You need to log in before you can comment on or make changes to this bug.