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