Martin Baker wrote:
>
> PS I do have some further changes to this file (comments and minor code
> changes) but I will send those separately as I think thats what you prefer.
Yes, that way it goes smoother.
--
Waldek Hebisch
--
You received this message because
>
> So only apply the change on array1.spad then?
I have applied it.
> One finaly reason to use SVREF: according to
> https://sourceforge.net/p/sbcl/mailman/message/30609918/
> it's 100x faster to compile SVREF than AREF.
I will look at this more. I see no difference in
build time, so impact
oldk1331 wrote:
> > - We probaly should be testing speed after doing:
> >
> > )lisp (proclaim '(optimize (speed 3) (safety 0)))
>
> I can't believe that this is not the default! Adding it
> to my ~/.fricas.init .
On testsuite safety 0 gives something like 10-20% speedup.
It is probably largest
Minor code changes to computation.spad
1) Change sayMsg to print.
2) Improve OutputForm, don't use string.
3) implement CoercibleTo(OutputForm).
new file is here:
https://github.com/martinbaker/multivector/blob/master/computation.spad
patch is here:
I have just commited patch which makes inlinig effective
in interpreter code. Now low level functions in interpreter
should run at similar speed as Spad code.
Note: by default interpter functions are compiled, while
command line is passed to Lisp interpreter. So loops
on command line are likely
Martin Baker wrote:
>
> Minor code changes to computation.spad
> 1) Change sayMsg to print.
> 2) Improve OutputForm, don't use string.
> 3) implement CoercibleTo(OutputForm).
I see that you have a lot of code like:
str::Symbol::OutputForm
If you want to print string without quotes use
This patch only modifies URLs, no other changes.
Spaces are removed and URLs updated, invalid URLs removed.
new file is here:
https://github.com/martinbaker/multivector/blob/master/computation.spad
and patch is here:
https://github.com/martinbaker/multivector/blob/master/computation.patch