From julian.rueth at fsfe.org Sat Jun 10 13:34:53 2017 From: julian.rueth at fsfe.org (Julian =?iso-8859-1?Q?R=FCth?=) Date: Sat, 10 Jun 2017 10:34:53 -0700 Subject: [Cython] Next Cython Release Message-ID: <20170610172533.GF2217@wigwum> Hello Cython developers, some people over at conda-forge would like to use features that are already in master but have not been released yet (https://github.com/conda-forge/cython-feedstock/pull/20). We would prefer not to patch and older release for Cython but just wait for the next release. However, since there has not been a release in 6 months, we're wondering if there are any plans for a release? julian PS: Not sure if this is the right list for this but the link to the bug-tracker on https://mail.python.org/mailman/listinfo/cython-devel goes to http://cython.org/trac/cython_trac/ which produces a 404. From nathan12343 at gmail.com Sun Jun 11 12:42:47 2017 From: nathan12343 at gmail.com (Nathan Goldbaum) Date: Sun, 11 Jun 2017 16:42:47 +0000 Subject: [Cython] Next Cython Release In-Reply-To: <20170610172533.GF2217@wigwum> References: <20170610172533.GF2217@wigwum> Message-ID: Cython uses github for bug tracking these days: https://github.com/cython/cython/issues On Sun, Jun 11, 2017 at 10:54 AM Julian R?th wrote: > Hello Cython developers, > > some people over at conda-forge would like to use features that are > already in master but have not been released yet > (https://github.com/conda-forge/cython-feedstock/pull/20). We would > prefer not to patch and older release for Cython but just wait for the > next release. However, since there has not been a release in 6 months, > we're wondering if there are any plans for a release? > > julian > > PS: Not sure if this is the right list for this but the link to the > bug-tracker on https://mail.python.org/mailman/listinfo/cython-devel > goes to http://cython.org/trac/cython_trac/ which produces a 404. > _______________________________________________ > cython-devel mailing list > cython-devel at python.org > https://mail.python.org/mailman/listinfo/cython-devel > -------------- next part -------------- An HTML attachment was scrubbed... URL: From robertwb at gmail.com Mon Jun 12 13:35:22 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Mon, 12 Jun 2017 10:35:22 -0700 Subject: [Cython] Next Cython Release In-Reply-To: <20170610172533.GF2217@wigwum> References: <20170610172533.GF2217@wigwum> Message-ID: On Sat, Jun 10, 2017 at 10:34 AM, Julian R?th wrote: > Hello Cython developers, > > some people over at conda-forge would like to use features that are > already in master but have not been released yet > (https://github.com/conda-forge/cython-feedstock/pull/20). We would > prefer not to patch and older release for Cython but just wait for the > next release. However, since there has not been a release in 6 months, > we're wondering if there are any plans for a release? I've been wanting to do a release for a while, just haven't gotten around to it. Yes, let's get one out soon. > PS: Not sure if this is the right list for this but the link to the > bug-tracker on https://mail.python.org/mailman/listinfo/cython-devel > goes to http://cython.org/trac/cython_trac/ which produces a 404. As mentioned, we're using github these days. I added redirects (including specific tickets like http://cython.org/trac/ticket/101 ) but seemed to have missed this link, thanks for pointing that out. From stefan_ml at behnel.de Mon Jun 12 14:00:17 2017 From: stefan_ml at behnel.de (Stefan Behnel) Date: Mon, 12 Jun 2017 20:00:17 +0200 Subject: [Cython] Next Cython Release In-Reply-To: References: <20170610172533.GF2217@wigwum> Message-ID: Robert Bradshaw schrieb am 12.06.2017 um 19:35: > On Sat, Jun 10, 2017 at 10:34 AM, Julian R?th wrote: >> some people over at conda-forge would like to use features that are >> already in master but have not been released yet >> (https://github.com/conda-forge/cython-feedstock/pull/20). We would >> prefer not to patch and older release for Cython but just wait for the >> next release. However, since there has not been a release in 6 months, >> we're wondering if there are any plans for a release? > > I've been wanting to do a release for a while, just haven't gotten > around to it. Yes, let's get one out soon. I've restarted working on the integration of PEP 525 and 530 (async generators/comprehensions), which gets closer but isn't quite there yet. Since we didn't have a release for so long now, and we certainly have enough on our current list of changes, I'd be ok with having it miss this release. So, +1 for a release soon. >> PS: Not sure if this is the right list for this but the link to the >> bug-tracker on https://mail.python.org/mailman/listinfo/cython-devel >> goes to http://cython.org/trac/cython_trac/ which produces a 404. > > As mentioned, we're using github these days. I added redirects > (including specific tickets like http://cython.org/trac/ticket/101 ) > but seemed to have missed this link, thanks for pointing that out. I already updated that link. Sorry, should have mentioned it. Stefan From nathan12343 at gmail.com Thu Jun 15 16:23:59 2017 From: nathan12343 at gmail.com (Nathan Goldbaum) Date: Thu, 15 Jun 2017 15:23:59 -0500 Subject: [Cython] Next Cython Release In-Reply-To: References: <20170610172533.GF2217@wigwum> Message-ID: Hearty +1 to getting a cython release out. I'd like to get this fix which will substantially speed up some code I've written on multicore systems: https://github.com/cython/cython/commit/8dbc7c39d1f1a3b675f69dceaac281 7b7675ca09 -Nathan On Mon, Jun 12, 2017 at 1:00 PM, Stefan Behnel wrote: > Robert Bradshaw schrieb am 12.06.2017 um 19:35: > > On Sat, Jun 10, 2017 at 10:34 AM, Julian R?th wrote: > >> some people over at conda-forge would like to use features that are > >> already in master but have not been released yet > >> (https://github.com/conda-forge/cython-feedstock/pull/20). We would > >> prefer not to patch and older release for Cython but just wait for the > >> next release. However, since there has not been a release in 6 months, > >> we're wondering if there are any plans for a release? > > > > I've been wanting to do a release for a while, just haven't gotten > > around to it. Yes, let's get one out soon. > > I've restarted working on the integration of PEP 525 and 530 (async > generators/comprehensions), which gets closer but isn't quite there yet. > Since we didn't have a release for so long now, and we certainly have > enough on our current list of changes, I'd be ok with having it miss this > release. So, +1 for a release soon. > > > >> PS: Not sure if this is the right list for this but the link to the > >> bug-tracker on https://mail.python.org/mailman/listinfo/cython-devel > >> goes to http://cython.org/trac/cython_trac/ which produces a 404. > > > > As mentioned, we're using github these days. I added redirects > > (including specific tickets like http://cython.org/trac/ticket/101 ) > > but seemed to have missed this link, thanks for pointing that out. > > I already updated that link. Sorry, should have mentioned it. > > Stefan > _______________________________________________ > cython-devel mailing list > cython-devel at python.org > https://mail.python.org/mailman/listinfo/cython-devel > -------------- next part -------------- An HTML attachment was scrubbed... URL: From jdemeyer at cage.ugent.be Fri Jun 16 03:59:59 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Fri, 16 Jun 2017 09:59:59 +0200 Subject: [Cython] Next Cython Release In-Reply-To: References: <20170610172533.GF2217@wigwum> Message-ID: <59438FFF.4080206@cage.ugent.be> On 2017-06-12 19:35, Robert Bradshaw wrote: > I've been wanting to do a release for a while, just haven't gotten > around to it. Yes, let's get one out soon. If you do, could you please consider https://github.com/cython/cython/pull/1654 or at least say why you don't like it. From adrien at guinet.me Sat Jun 17 05:39:22 2017 From: adrien at guinet.me (Adrien Guinet) Date: Sat, 17 Jun 2017 11:39:22 +0200 Subject: [Cython] Next Cython Release In-Reply-To: <59438FFF.4080206@cage.ugent.be> References: <20170610172533.GF2217@wigwum> <59438FFF.4080206@cage.ugent.be> Message-ID: <00b1ff3d-92aa-e9ab-2f89-d3b3c1b0714f@guinet.me> On 06/16/2017 09:59 AM, Jeroen Demeyer wrote: > On 2017-06-12 19:35, Robert Bradshaw wrote: >> I've been wanting to do a release for a while, just haven't gotten >> around to it. Yes, let's get one out soon. > > If you do, could you please consider > https://github.com/cython/cython/pull/1654 > > or at least say why you don't like it. As everyone is making his "wishlist" for the next Cython release, I think it could be nice if https://github.com/cython/cython/pull/1607 (Pythran support for Numpy operation) could be integrated. I just pushed a commit that makes the usage easier, just need travis to confirme and squash everything to a single commit. Please let me know if something could be blocking from your side. Thanks! Regards, -- Adrien. From robertwb at gmail.com Mon Jun 19 17:20:24 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Mon, 19 Jun 2017 14:20:24 -0700 Subject: [Cython] Cython 0.26 release Message-ID: I'd like to officially start off the release process with an alpha candidate. This does necessarily contain everything that will be in the final release, but given the amount of time that has passed I would recommend everyone test this out with their projects sooner rather than later. Download: https://github.com/cython/cython/archive/0.26.alpha0.zip Initial changelog: https://github.com/cython/cython/blob/0.26.alpha0/CHANGES.rst Please feel free to respond to this thread or file bugs as necessary. Thanks, Robert From jdemeyer at cage.ugent.be Tue Jun 20 10:53:32 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Tue, 20 Jun 2017 16:53:32 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: Message-ID: <594936EC.6070701@cage.ugent.be> This is breaking SageMath: https://github.com/cython/cython/issues/1739 From robertwb at math.washington.edu Tue Jun 20 16:02:27 2017 From: robertwb at math.washington.edu (Robert Bradshaw) Date: Tue, 20 Jun 2017 13:02:27 -0700 Subject: [Cython] Cython 0.26 release In-Reply-To: <594936EC.6070701@cage.ugent.be> References: <594936EC.6070701@cage.ugent.be> Message-ID: Thanks. Fixed. On Tue, Jun 20, 2017 at 7:53 AM, Jeroen Demeyer wrote: > This is breaking SageMath: > https://github.com/cython/cython/issues/1739 > _______________________________________________ > cython-devel mailing list > cython-devel at python.org > https://mail.python.org/mailman/listinfo/cython-devel From jdemeyer at cage.ugent.be Wed Jun 21 08:29:14 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Wed, 21 Jun 2017 14:29:14 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: Message-ID: <594A669A.7050505@cage.ugent.be> Could the error from https://github.com/cython/cython/issues/1732 at https://github.com/cython/cython/blob/master/Cython/Compiler/Symtab.py#L2099 be a warning instead? SageMath has quite a bit of code hitting this error. It would make our life easier if we could fix it gradually instead of being forced to fix it all before upgrading to Cython 0.26. From jdemeyer at cage.ugent.be Wed Jun 21 08:50:32 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Wed, 21 Jun 2017 14:50:32 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: Message-ID: <594A6B98.8050101@cage.ugent.be> I'm getting an infinite recursion in deepcopy() in fresh_finally_clause() here: https://github.com/cython/cython/blob/master/Cython/Compiler/Nodes.py#L7022 I'm reporting it now in case you immediately know what the problem is. I'll try to analyse it later. From jdemeyer at cage.ugent.be Wed Jun 21 09:55:13 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Wed, 21 Jun 2017 15:55:13 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: <594A6B98.8050101@cage.ugent.be> References: <594A6B98.8050101@cage.ugent.be> Message-ID: <594A7AC1.9090803@cage.ugent.be> See https://github.com/cython/cython/issues/1744 This seems to be the final issue breaking SageMath with Cython master. From robertwb at gmail.com Wed Jun 21 11:44:48 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Wed, 21 Jun 2017 08:44:48 -0700 Subject: [Cython] Cython 0.26 release In-Reply-To: <594A669A.7050505@cage.ugent.be> References: <594A669A.7050505@cage.ugent.be> Message-ID: On Wed, Jun 21, 2017 at 5:29 AM, Jeroen Demeyer wrote: > Could the error from > https://github.com/cython/cython/issues/1732 > at > https://github.com/cython/cython/blob/master/Cython/Compiler/Symtab.py#L2099 > be a warning instead? > > SageMath has quite a bit of code hitting this error. It would make our life > easier if we could fix it gradually instead of being forced to fix it all > before upgrading to Cython 0.26. Could you give some examples? Is it cdef->cpdef, or something else? It's possible that the error needs to be made narrower, but I'd really like to avoid incorrect code that could lead to wrong methods being called. From jdemeyer at cage.ugent.be Wed Jun 21 12:59:48 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Wed, 21 Jun 2017 18:59:48 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: <594A669A.7050505@cage.ugent.be> Message-ID: <594AA604.9040805@cage.ugent.be> On 2017-06-21 17:44, Robert Bradshaw wrote: > Is it cdef->cpdef, or something else? Yes, it is. > It's possible that the error needs to be made narrower, but I'd really > like to avoid incorrect code that could lead to wrong methods being > called. Fine... we can still patch Cython in Sage. From insertinterestingnamehere at gmail.com Wed Jun 21 13:42:06 2017 From: insertinterestingnamehere at gmail.com (Ian Henriksen) Date: Wed, 21 Jun 2017 17:42:06 +0000 Subject: [Cython] Cython 0.26 release In-Reply-To: References: Message-ID: On Mon, Jun 19, 2017 at 4:27 PM Robert Bradshaw wrote: > I'd like to officially start off the release process with an alpha > candidate. This does necessarily contain everything that will be in > the final release, but given the amount of time that has passed I > would recommend everyone test this out with their projects sooner > rather than later. > > Download: https://github.com/cython/cython/archive/0.26.alpha0.zip > Initial changelog: > https://github.com/cython/cython/blob/0.26.alpha0/CHANGES.rst > > Please feel free to respond to this thread or file bugs as necessary. > > Thanks, > Robert > I ran a quick test against dynd-python with master this morning. Everything looks fine there. Thanks for managing the release. Best, Ian Henriksen -------------- next part -------------- An HTML attachment was scrubbed... URL: From robertwb at gmail.com Wed Jun 21 14:25:36 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Wed, 21 Jun 2017 11:25:36 -0700 Subject: [Cython] Cython 0.26 release In-Reply-To: <594AA604.9040805@cage.ugent.be> References: <594A669A.7050505@cage.ugent.be> <594AA604.9040805@cage.ugent.be> Message-ID: On Wed, Jun 21, 2017 at 9:59 AM, Jeroen Demeyer wrote: > On 2017-06-21 17:44, Robert Bradshaw wrote: >> >> Is it cdef->cpdef, or something else? > > Yes, it is. > >> It's possible that the error needs to be made narrower, but I'd really >> like to avoid incorrect code that could lead to wrong methods being >> called. > > Fine... we can still patch Cython in Sage. I suppose in that case we just got lucky that no one cimported these classes and tried to use methods after this point in the vtable... From jdemeyer at cage.ugent.be Wed Jun 21 15:41:16 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Wed, 21 Jun 2017 21:41:16 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: <594A669A.7050505@cage.ugent.be> <594AA604.9040805@cage.ugent.be> Message-ID: <594ACBDC.4000107@cage.ugent.be> On 2017-06-21 20:25, Robert Bradshaw wrote: > I suppose in that case we just got lucky that no one cimported these > classes and tried to use methods after this point in the vtable... I think that's the case indeed. From robertwb at gmail.com Wed Jun 21 15:51:14 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Wed, 21 Jun 2017 12:51:14 -0700 Subject: [Cython] Cython 0.26 release In-Reply-To: <594ACBDC.4000107@cage.ugent.be> References: <594A669A.7050505@cage.ugent.be> <594AA604.9040805@cage.ugent.be> <594ACBDC.4000107@cage.ugent.be> Message-ID: On Wed, Jun 21, 2017 at 12:41 PM, Jeroen Demeyer wrote: > On 2017-06-21 20:25, Robert Bradshaw wrote: >> >> I suppose in that case we just got lucky that no one cimported these >> classes and tried to use methods after this point in the vtable... > > I think that's the case indeed. I'll see if I can put together a patch. From jdemeyer at cage.ugent.be Thu Jun 22 04:25:13 2017 From: jdemeyer at cage.ugent.be (Jeroen Demeyer) Date: Thu, 22 Jun 2017 10:25:13 +0200 Subject: [Cython] Cython 0.26 release In-Reply-To: References: <594A669A.7050505@cage.ugent.be> <594AA604.9040805@cage.ugent.be> <594ACBDC.4000107@cage.ugent.be> Message-ID: <594B7EE9.8010706@cage.ugent.be> On 2017-06-21 21:51, Robert Bradshaw wrote: > I'll see if I can put together a patch. For what? For Sage? For Cython? Anyway, I think that coping with this new error message is not particularly hard, just a lot of work. That's why I would prefer a warning instead of an error. That way, we can easily see how bad the problem is and what needs to be fixed and we fix it gradually (say, before Cython 0.27 which could make this a hard error). From robertwb at gmail.com Fri Jun 23 02:18:18 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Thu, 22 Jun 2017 23:18:18 -0700 Subject: [Cython] On cdef and cpdef discrepencies Message-ID: On Thu, Jun 22, 2017 at 1:25 AM, Jeroen Demeyer wrote: > On 2017-06-21 21:51, Robert Bradshaw wrote: >> >> I'll see if I can put together a patch. > > > For what? For Sage? For Cython? I was thinking for Sage. I was assuming there was just an oddball collection of oversights accumulated over the yeras. > Anyway, I think that coping with this new error message is not particularly > hard, just a lot of work. That's why I would prefer a warning instead of an > error. That way, we can easily see how bad the problem is and what needs to > be fixed and we fix it gradually (say, before Cython 0.27 which could make > this a hard error). I was digging into this a bit more, and the issue seems to be entirely the arithmetic methods like _add_, and was introduced one year and one day ago: https://github.com/sagemath/sage/commit/e1d2ba46574273ec4d358e654ed195a82b81a654 . It'd be cleaner if these were in fact cpdef methods, but our category method injection framework requires that the class not have an _add_ defined at the Python level. But any overrides must be cpdef methods lest a (Python) call to x._add_(y) succeed but not find the override. Perhaps it'd be more clear to define cpdef Element._add_ whose implementation directly calls getattr_from_category rather than (self)._add_. - Robert From roed.math at gmail.com Fri Jun 23 21:32:45 2017 From: roed.math at gmail.com (David Roe) Date: Fri, 23 Jun 2017 18:32:45 -0700 Subject: [Cython] Cython 0.26 release In-Reply-To: <594ACBDC.4000107@cage.ugent.be> References: <594A669A.7050505@cage.ugent.be> <594AA604.9040805@cage.ugent.be> <594ACBDC.4000107@cage.ugent.be> Message-ID: On Wed, Jun 21, 2017 at 12:41 PM, Jeroen Demeyer wrote: > On 2017-06-21 20:25, Robert Bradshaw wrote: > >> I suppose in that case we just got lucky that no one cimported these >> classes and tried to use methods after this point in the vtable... >> > > I think that's the case indeed. In fact, Julian Rueth and I eventually did cimport one of these classes (Polynomial_generic_dense), eventually leading to https://github.com/cython/cython/issues/1732. A warning does seem like it would make fixing existing code easier. David > > > _______________________________________________ > cython-devel mailing list > cython-devel at python.org > https://mail.python.org/mailman/listinfo/cython-devel > -------------- next part -------------- An HTML attachment was scrubbed... URL: From stefan_ml at behnel.de Sun Jun 25 17:58:28 2017 From: stefan_ml at behnel.de (Stefan Behnel) Date: Sun, 25 Jun 2017 23:58:28 +0200 Subject: [Cython] Next Cython Release In-Reply-To: <00b1ff3d-92aa-e9ab-2f89-d3b3c1b0714f@guinet.me> References: <20170610172533.GF2217@wigwum> <59438FFF.4080206@cage.ugent.be> <00b1ff3d-92aa-e9ab-2f89-d3b3c1b0714f@guinet.me> Message-ID: <0e0841b7-9212-67fe-6a9f-2d054e7ef12c@behnel.de> Adrien Guinet schrieb am 17.06.2017 um 11:39: > As everyone is making his "wishlist" for the next Cython release, I > think it could be nice if https://github.com/cython/cython/pull/1607 > (Pythran support for Numpy operation) could be integrated. I just pushed > a commit that makes the usage easier, just need travis to confirme and > squash everything to a single commit. Looks good to me now. I'll leave it to Robert to decide if this should be included in this release. I think it's ready, but I'm not in the target group and haven't tested it myself. Stefan From robertwb at gmail.com Tue Jun 27 02:34:25 2017 From: robertwb at gmail.com (Robert Bradshaw) Date: Mon, 26 Jun 2017 23:34:25 -0700 Subject: [Cython] Next Cython Release In-Reply-To: <0e0841b7-9212-67fe-6a9f-2d054e7ef12c@behnel.de> References: <20170610172533.GF2217@wigwum> <59438FFF.4080206@cage.ugent.be> <00b1ff3d-92aa-e9ab-2f89-d3b3c1b0714f@guinet.me> <0e0841b7-9212-67fe-6a9f-2d054e7ef12c@behnel.de> Message-ID: I'm happy with including it at this early stage. On Sun, Jun 25, 2017 at 2:58 PM, Stefan Behnel wrote: > Adrien Guinet schrieb am 17.06.2017 um 11:39: >> As everyone is making his "wishlist" for the next Cython release, I >> think it could be nice if https://github.com/cython/cython/pull/1607 >> (Pythran support for Numpy operation) could be integrated. I just pushed >> a commit that makes the usage easier, just need travis to confirme and >> squash everything to a single commit. > > Looks good to me now. I'll leave it to Robert to decide if this should be > included in this release. I think it's ready, but I'm not in the target > group and haven't tested it myself. > > Stefan > _______________________________________________ > cython-devel mailing list > cython-devel at python.org > https://mail.python.org/mailman/listinfo/cython-devel