Let me know if you want me to break anything else. My mom always hated it when I was a kid, but it seems to be useful on pre-releases.<br><br><div class="gmail_quote">On Sun, Oct 9, 2011 at 7:46 PM, Matthew Flatt <span dir="ltr">&lt;<a href="mailto:mflatt@cs.utah.edu">mflatt@cs.utah.edu</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Fixed.<br>
<br>
Some unsafe primitives were marked internally as &quot;functional&quot;, but this<br>
annotation was used with two different interpretations: sometimes as<br>
&quot;non-mutating&quot;, sometimes as &quot;always produces the same result&quot;. The<br>
`unsafe-vector-ref&#39; primitive was marked as &quot;functional&quot; with the<br>
former intent, but the compiler could use the latter interpretation to<br>
move it past an `unsafe-vector-set!&#39;. The solution, of course, is to<br>
have two distinct annotations.<br>
<div><div></div><div class="h5"><br>
At Sun, 9 Oct 2011 15:02:13 -0600, Doug Williams wrote:<br>
&gt; I reloaded the older version of Racket on my Windows 7 computer and the<br>
&gt; radix-2 FFTs run fine there. I also tested both versions on my Macbook Pro<br>
&gt; (32 bit) and got the same behavior - correct results (plots) on 5.1.2 and<br>
&gt; bad radix-2 results (plots) on 5.1.900.1.<br>
&gt; code for the mixed-radix and radix-2. So, it isn&#39;t limited to Windows 7 or<br>
&gt; 64-bit.<br>
&gt;<br>
&gt; Doug<br>
&gt;<br>
&gt; On Sun, Oct 9, 2011 at 1:44 PM, Doug Williams<br>
&gt; &lt;<a href="mailto:m.douglas.williams@gmail.com">m.douglas.williams@gmail.com</a>&gt;wrote:<br>
&gt;<br>
&gt; &gt; I downloaded the latest pre-release version (5.1.900.1) to update the<br>
&gt; &gt; science collection to use the new plot collection. The good news is that<br>
&gt; &gt; that went very smoothly. The bad news is that some of my FFT routines seem<br>
&gt; &gt; to be getting incorrect numeric results - at least the plots are very bad.<br>
&gt; &gt; But, since all of the other plots seem fine, I don&#39;t see why these would be<br>
&gt; &gt; any different.<br>
&gt; &gt;<br>
&gt; &gt; So, I suspect that something has changed that affects the numeric<br>
&gt; &gt; calculations. It is just the radix-2 FFTs that are having the problem. They<br>
&gt; &gt; do some low-level bit fiddling to do the in-place butterfly addressing for<br>
&gt; &gt; the FFTs - using things like unsafe-fxlshift (which just looks unfriendly) -<br>
&gt; &gt; so, I suspect something there. Are the any recent changes that would affect<br>
&gt; &gt; these kinds of fixed-point operations?<br>
&gt; &gt;<br>
&gt; &gt; I&#39;ve run the code under a previous version of Racket on a 64-bit Linux<br>
&gt; &gt; (Scientific Linux 6.0) computer and a 32-bit Windows XP computer and get<br>
&gt; &gt; correct results. So, I don&#39;t think it is simply a 32/64 bit problem - unless<br>
&gt; &gt; it is limited to 64-bit Windows.<br>
&gt; &gt;<br>
&gt; &gt; Sorry that was kind of rambling and non-specific, but I was wondering where<br>
&gt; &gt; to start looking at the problem or what would help someone else look into<br>
&gt; &gt; it.<br>
&gt; &gt;<br>
&gt; &gt; Doug<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; On Fri, Oct 7, 2011 at 10:19 PM, Ryan Culpepper &lt;<a href="mailto:ryan@cs.utah.edu">ryan@cs.utah.edu</a>&gt; wrote:<br>
&gt; &gt;<br>
&gt; &gt;&gt; The release process for v5.2 has begun: the `release&#39; branch was<br>
&gt; &gt;&gt; created for any work that is left and is now bumped to v5.1.90.  You<br>
&gt; &gt;&gt; can go on using the `master&#39; branch as usual, it is now bumped to<br>
&gt; &gt;&gt; v5.2.0.1 (to avoid having two different trees with the same version).<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; If you have any bug-fixes and changes that need to go in the release<br>
&gt; &gt;&gt; then make sure to specify that in the commit message or mail me the<br>
&gt; &gt;&gt; commit SHA1s.  You can `git checkout release&#39; to try it out directly if<br>
&gt; &gt;&gt; needed -- but do not try to push commits on it (the server will forbid<br>
&gt; &gt;&gt; it).<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Please make sure that code that you&#39;re responsible for is as stable<br>
&gt; &gt;&gt; as possible, and let me know if there is any new work that should<br>
&gt; &gt;&gt; not be included in this release.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;      &gt;&gt; NOW IS THE TIME TO FIX BUGS THAT YOU KNOW ABOUT &lt;&lt;&lt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; The time between the `release&#39; branch creation and the actual<br>
&gt; &gt;&gt; release is for fixing new errors that prevent proper functioning of<br>
&gt; &gt;&gt; major components and that show up during the preparation for a<br>
&gt; &gt;&gt; release.  You can also finalize piece of work that is not yet<br>
&gt; &gt;&gt; complete, but please avoid merging new features.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Note that nightly builds will go on as usual (as v5.2.0.1), and<br>
&gt; &gt;&gt; pre-release builds will be available shortly at<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;  <a href="http://pre.racket-lang.org/release/" target="_blank">http://pre.racket-lang.org/release/</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Please tell me if you think that this release is significant enough<br>
&gt; &gt;&gt; that it should be announced on the users list for wider testing.<br>
&gt; &gt;&gt; --<br>
&gt; &gt;&gt; Ryan Culpepper<br>
&gt; &gt;&gt; _________________________________________________<br>
&gt; &gt;&gt;  For list-related administrative tasks:<br>
&gt; &gt;&gt;  <a href="http://lists.racket-lang.org/listinfo/dev" target="_blank">http://lists.racket-lang.org/listinfo/dev</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; _________________________________________________<br>
&gt;   For list-related administrative tasks:<br>
&gt;   <a href="http://lists.racket-lang.org/listinfo/dev" target="_blank">http://lists.racket-lang.org/listinfo/dev</a><br>
</div></div></blockquote></div><br>