[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|---

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #30 from Fed

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED -- You are receiving

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #29 from Fedora Update System --- nest-2.14.0-4.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2018-b95878833b -- You are receiving this mail because: You are on the CC list for

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #28 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/nest -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about ch

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #27 from Ankur Sinha (FranciscoD) --- Thanks for the review! I'll leave all arches enabled then. It'll take a little longer to build, but that's fine. I've noticed that libneurosim is an optional dependency that NEST needs to be

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Status|ASSIGNED|POST Flags|

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Ankur Sinha (FranciscoD) changed: What|Removed |Added Blocks||1276941 (fedora-neuro) Re

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Ankur Sinha (FranciscoD) changed: What|Removed |Added Blocks||1150105 Referenced Bugs:

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #25 from Ankur Sinha (FranciscoD) --- F28 scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=28651669 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about chan

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #24 from Ankur Sinha (FranciscoD) --- Hi! I've made the required changes. It builds, and I've run a scratch build for F28 and tested the packages too. I've also added a README.Fedora file to explain how the package should be used.

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #23 from Zbigniew Jędrzejewski-Szmek --- So luckily there's no conflict over /usr/bin/nest. Nestopia uses /usr/bin/nestopia and nested uses /usr/bin/nested. Similarly for /usr/bin/sli, there are a few similar names, but nothing exa

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Status|NEW |ASSIGNED Assign

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #22 from Zbigniew Jędrzejewski-Szmek --- > # They run their tests per commit, so we don't need to run them here to check > # it all again. That's almost certainly wrong. Upstream does not test the same combination of architectures,

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Ankur Sinha (FranciscoD) changed: What|Removed |Added Whiteboard|NOTREADY| --- Comment #21 from Anku

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #20 from Zbigniew Jędrzejewski-Szmek --- Python .so files have rpath set, you'll need to get rid of it [https://fedoraproject.org/wiki/Packaging:Guidelines#Beware_of_Rpath]. I haven't look here, but sometimes -DCMAKE_SKIP_RPATH:BOO

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #19 from Ankur Sinha (FranciscoD) --- Oh, by the way, this version gets the latest cmake version of the code from this pull request: https://github.com/nest/nest-simulator/pull/318 I've put up the srpm too: https://ankursinha.fedo

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #18 from Ankur Sinha (FranciscoD) --- This is the current version, which doesn't quite work: https://ankursinha.fedorapeople.org/misc/nest.spec They use cmake, which builds the c++ bits to build the SLI version. The cmake then ca

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #17 from Zbigniew Jędrzejewski-Szmek --- I'm don't think you need to build 6 times. The python wrapper might be independent of mpi. Can you post your latest version of the spec file, I'll try to have a look. If the python module (.

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #16 from Ankur Sinha (FranciscoD) --- So, I've started working on the cmake spec. I have a question though - I need to build it without mpi, with mpich, with openmpi, and for each of these 3, with python2 and python3 - so, can I ju

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #15 from Ankur Sinha (FranciscoD) --- Pull request with cmake build system: https://github.com/nest/nest-simulator/pull/213 I suggest we hold off until this is merged? -- You are receiving this mail because: You are on the CC li

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #14 from Zbigniew Jędrzejewski-Szmek --- (In reply to Ankur Sinha (FranciscoD) from comment #13) > (In reply to Zbigniew Jędrzejewski-Szmek from comment #12) > > (In reply to Ankur Sinha (FranciscoD) from comment #11) > > > I'm hap

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #13 from Ankur Sinha (FranciscoD) --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #12) > (In reply to Ankur Sinha (FranciscoD) from comment #11) > > I'm happy to continue the review if another reviewer comes along, > I do

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #12 from Zbigniew Jędrzejewski-Szmek --- (In reply to Ankur Sinha (FranciscoD) from comment #11) > I'm happy to continue the review if another reviewer comes along, I don't understand this sentence. Raphael said that he can review

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #11 from Ankur Sinha (FranciscoD) --- It isn't actually a hypothetical upstream change. The nest user mailing list has received clear emails that detail the move to NEST3, which they say is a complete overhaul of the code base - wh

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #10 from Zbigniew Jędrzejewski-Szmek --- > do you think it makes sense to hold off the review until they've completed > the change? I don't think that waiting for some hypothetical future upstream changes makes much sense. At mos

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Ankur Sinha (FranciscoD) changed: What|Removed |Added Whiteboard||NOTREADY -- You are recei

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #9 from Ankur Sinha (FranciscoD) --- Heya, So, upstream confirmed that nest supports py3, but they've also said that they intend to redo the build system in the future. If the build system is redone, I'll have to pretty much redo

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #8 from Raphael Groner --- For a first step, I found some general advices to your spec file. MUST: - Add full URL to Source0, it's not sufficient to mention in comment only. I guess you use the snapshot tarball of GitHub, then p

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #7 from Ankur Sinha (FranciscoD) --- Taken them up. These are the latest spec/srp links: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest.spec https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #6 from Raphael Groner --- Review swap maybe with mdp (bug #1246790) or polyglot (bug #1197333)? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product an

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 Raphael Groner changed: What|Removed |Added CC||projects...@smart.ms --- Comment #5

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #4 from Ankur Sinha (FranciscoD) --- New srpm: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest-2.8.0-4.gitdc21fdc.fc23.src.rpm Spec: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest.spec --

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #3 from Ankur Sinha (FranciscoD) --- New srpm: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest-2.8.0-3.gitdc21fdc.fc23.src.rpm Spec: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest.spec --

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #2 from Ankur Sinha (FranciscoD) --- Updated srpm with improvements: https://ankursinha.fedorapeople.org/neuroscience-research-copr/nest-2.8.0-2.gitdc21fdc.fc23.src.rpm Spec: https://ankursinha.fedorapeople.org/neuroscience-resear

[Bug 1273579] Review Request: nest - The neural simulation tool

https://bugzilla.redhat.com/show_bug.cgi?id=1273579 --- Comment #1 from Ankur Sinha (FranciscoD) --- Copr builds here: https://copr.fedoraproject.org/coprs/ankursinha/neuroscience-research/package/nest/ -- You are receiving this mail because: You are on the CC list for the bug. You are alway