#7013: [with patch, needs work] prime_pi and nth_prime
---------------------------+------------------------------------------------
 Reporter:  kevin.stueve   |       Owner:  kevin.stueve            
     Type:  enhancement    |      Status:  new                     
 Priority:  major          |   Milestone:                          
Component:  number theory  |    Keywords:  primes, sieve, table,LMO
 Reviewer:  was            |      Author:  Kevin Stueve            
   Merged:                 |  
---------------------------+------------------------------------------------

Comment(by GeorgSWeber):

 Hi Kevin,

 you posted a *.lzma-compressed file. On my Mac, neither 7-zip installs out
 of the box (seems to be MS Windows-only, see http://www.7-zip.org/), nor
 do I intend to install DarwinPorts or Fink, which contain packages to
 unpack such files, nor did I find some standalone *.dmg for OS X by
 googling some minutes around. So I'm unable to unpack this file on my
 computer.

 Sorry, but that's a problem of your patch, not of my system installation.
 Patches in trac are supposed to be created by hg/mercurial and to apply
 seamlessly to Sage, or you should provide a link (!) to some file in
 *.spkg format, or both. Did you read the respective parts of the "Sage
 Programming Guide", by the way?

 From what you write in the sage-devel thread ("Be sure to use the -m32
 option when compiling the c code ...") it seems that (some C part of) your
 code lacks anyway build scripts that work "out of the box" on each of the
 platforms supported by Sage. For a start, have a look at the
 "Cliquer.spkg" already in Sage, or at the "Frobby.spkg", not yet in Sage
 due to build problems on Solaris (see trac ticket #6416), in case your C
 code is rather "external" to the Sage library.

 On the other hand, if your code is intended to "go directly into the Sage
 library", you need anyway to tell the then-necessary Cython bridge how to
 compile your code correctly (see module_list.py). And probably some
 database integration, if I understand correctly what you want to do.

 Better still, drop the necessity of "-m32" by using, in your code, fixed-
 sized C types like "uint32_t" from (ISO/ANSI C99) stdlib.h (wich exists
 even for msvc, although provided by third parties), and not plain
 "int"/"long"/... if your algorithm depends on that. But without being able
 to even have a single look at your code, I'll stop here.

 And please, please, don't post MB-sized compressed files into trac itself,
 but rather post links to some accessible webspace instead. I did that
 myself for one of my first patches for Sage, and got pretty washed my head
 (see trac ticket #4857, if you want to), the file itself being erased from
 trac by the admin almost immediately ... (the reason is that trac is best
 used "text-based", and any compressed blob has to remain in there for
 eternity, which is bad for both size-archive-backup issues, and
 performance).

 Heads up! ;-)


 Cheers,
 Georg

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7013#comment:2>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to