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 1923779
| Summary: | Review Request: catalyst - API specification for simulations to analyze and visualize data in situ | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Orion Poplawski <orion> |
| Component: | Package Review | Assignee: | Zbigniew Jędrzejewski-Szmek <zbyszek> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | package-review, zbyszek |
| Target Milestone: | --- | Flags: | zbyszek:
fedora-review+
|
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2021-02-23 03:18:43 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 1923074 | ||
|
Description
Orion Poplawski
2021-02-01 21:45:27 UTC
> Originally, Catalyst was part of ParaView. However, with ParaView version > 5.9, the Catalyst API was revamped quite extensively and release as an > independent project. This repository includes the definition together with a > lightweight implementation of this revamped Catalyst API. > > The documentation describing the details, including build instructions and > how to use Catalyst for in situ analysis and visualization in simulations is > available here: https://catalyst-in-situ.readthedocs.io/en/latest/index.html "d" is missing. But alltogether, this is not very intesting for users. Maybe: Catalyst has been split out of ParaView. This package includes the definition together with a lightweight implementation of this Catalyst API. For details how to use Catalyst for in situ analysis and visualization in simulations, see https://catalyst-in-situ.readthedocs.io/en/latest/index.html. + pakcage name is OK + license is acceptable for Fedora (BSD 3-clause) + license is specified correctly + builds and installs OK + BR look correct + P:bundled() is used where appropriate + P/R look correct rpmlint: catalyst.src: W: spelling-error Summary(en_US) situ -> sit, suit, site catalyst.src: W: spelling-error %description -l en_US situ -> sit, suit, site catalyst.src: W: strange-permission catalyst-2fc94c55ba4eb994f27728141ebcf15c3435f306.tar.gz 660 catalyst.src: W: strange-permission catalyst-unbundle.patch 660 catalyst.src: W: strange-permission catalyst.spec 660 catalyst.src:24: W: unversioned-explicit-provides bundled(conduit) catalyst.x86_64: W: spelling-error Summary(en_US) situ -> sit, suit, site catalyst.x86_64: W: spelling-error %description -l en_US situ -> sit, suit, site catalyst.x86_64: W: no-documentation catalyst-devel.x86_64: W: no-documentation 5 packages and 0 specfiles checked; 0 errors, 10 warnings. Nothing important here. Package is APPROVED. Oh, please add %license License.txt 3rdPartyLicenses.txt Thanks for the review, will make the changes. (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/catalyst Checked in and built. |