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 1293043 (python-epac)

Summary: Review Request: python-epac - Machine learning workflow builder
Product: [Fedora] Fedora Reporter: Ilia Gradina <ilya.gradina>
Component: Package ReviewAssignee: Igor Raits <igor.raits>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: igor.raits, ilya.gradina, package-review, sanjay.ankur, zbyszek
Target Milestone: ---Flags: ignatenko: fedora-review?
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-07-10 09:58:31 UTC Type: ---
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: 201449, 1276941    

Description Ilia Gradina 2015-12-19 15:31:54 UTC
Spec URL: https://github.com/neurofedora/python-epac/raw/master/python-epac.spec
SRPM URL: https://github.com/neurofedora/python-epac/raw/master/python-epac-0.0.1-1.git45e63f2.fc23.src.rpm
Description: Embarrassingly Parallel Array Computing: EPAC is a machine learning workflow builder. 
Fedora Account System Username: ilgrad

Comment 1 Igor Gnatenko 2015-12-19 16:11:42 UTC
Missing requires, missing buildrequires, no python3 package. Binary should point to py3 version.

Comment 2 Zbigniew Jędrzejewski-Szmek 2015-12-20 00:41:28 UTC
fedora-review- is for packages which can't be fixed, while the review is in progress it's customary not to set anything, or to set fedora-review?.

Do not repeat the package name in the summary (it should be short to display properly in various listings). Maybe
Summary:  Machine learning workflow builder

The license tag for bsd 3-clause is just "BSD". See https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Good_Licenses.

You should create a python2 subpackage, and add support for python3 if possible.
See https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file for the new python packaging.

Comment 3 Igor Gnatenko 2016-08-08 05:56:55 UTC
ping?

Comment 4 Igor Gnatenko 2017-06-21 15:04:53 UTC
ping once more?

Comment 5 Ankur Sinha (FranciscoD) 2018-08-05 09:48:53 UTC
Hi Ilya,

Would you like to continue the review so that the package can be included in NeuroFedora?

Cheers!
Ankur

Comment 6 Ilia Gradina 2018-08-07 21:30:21 UTC
(In reply to Ankur Sinha (FranciscoD) from comment #5)
> Hi Ilya,
> 
> Would you like to continue the review so that the package can be included in
> NeuroFedora?
> 
> Cheers!
> Ankur

Hi Ankur,
yes, I  will continue the review.

Comment 7 Package Review 2020-07-10 00:54:13 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 8 Ankur Sinha (FranciscoD) 2020-07-10 09:58:31 UTC
Closing this now.