diff --git a/.gitignore b/.gitignore index d51d954cacc..1cb7b37243b 100644 --- a/.gitignore +++ b/.gitignore @@ -59,8 +59,6 @@ coverage_html_report/ # Project specific statsmodels/version.py -cythonize.dat -cythonize_exclusions.dat statsmodels.egg-info/ iterate.dat hash_dict.pickle diff --git a/MANIFEST.in b/MANIFEST.in index 520f1d5b0b8..5e350bb77f4 100644 --- a/MANIFEST.in +++ b/MANIFEST.in @@ -7,7 +7,6 @@ recursive-exclude dist * recursive-exclude tools * include tools/examples_rst.py include tools/hash_funcs.py -include tools/cythonize.py graft statsmodels/datasets graft statsmodels/sandbox/regression/data @@ -26,9 +25,6 @@ include docs/Makefile recursive-include examples * -# Cached Cython signatures -include cythonize.dat - prune */__pycache__ global-exclude *~ *.swp *.pyc *.pyo *.bak diff --git a/tools/cythonize.py b/tools/cythonize.py deleted file mode 100755 index 52fccda5a98..00000000000 --- a/tools/cythonize.py +++ /dev/null @@ -1,226 +0,0 @@ -#!/usr/bin/env python -""" cythonize - -Cythonize pyx files into C files as needed. - -Usage: cythonize [root_dir] - -Default [root_dir] is 'statsmodels'. - -Checks pyx files to see if they have been changed relative to their -corresponding C files. If they have, then runs cython on these files to -recreate the C files. - -The script thinks that the pyx files have changed relative to the C files -by comparing hashes stored in a database file. - -Simple script to invoke Cython (and Tempita) on all .pyx (.pyx.in) -files; while waiting for a proper build system. Uses file hashes to -figure out if rebuild is needed. - -For now, this script should be run by developers when changing Cython files -only, and the resulting C files checked in, so that end-users (and Python-only -developers) do not get the Cython/Tempita dependencies. - -Originally written by Dag Sverre Seljebotn, and copied here from: - -https://raw.githubusercontent.com/dagss/private-scipy-refactor/cythonize/cythonize.py - -Note: this script does not check any of the dependent C libraries; it only -operates on the Cython .pyx files. -""" - -from __future__ import division, print_function, absolute_import - -import os -import re -import sys -import hashlib -import subprocess - -HASH_FILE = 'cythonize.dat' -EXCLUSION_FILE = 'cythonize_exclusions.dat' -DEFAULT_ROOT = 'statsmodels' - -# WindowsError is not defined on unix systems -try: - WindowsError -except NameError: - WindowsError = None - - -# -# Rules -# -def process_pyx(fromfile, tofile): - try: - from Cython.Compiler.Version import version as cython_version - from distutils.version import LooseVersion - if LooseVersion(cython_version) < LooseVersion('0.19'): - raise Exception('Building Statsmodels requires Cython >= 0.19') - - except ImportError: - pass - - flags = ['--fast-fail'] - if tofile.endswith('.cxx'): - flags += ['--cplus'] - - try: - try: - r = subprocess.call(['cython'] + flags + ["-o", tofile, fromfile]) - if r != 0: - raise Exception('Cython failed') - except OSError: - # There are ways of installing Cython that don't result in a cython - # executable on the path, see gh-2397. - r = subprocess.call([sys.executable, '-c', - 'import sys; from Cython.Compiler.Main import ' - 'setuptools_main as main; sys.exit(main())'] + flags + - ["-o", tofile, fromfile]) - if r != 0: - raise Exception('Cython failed') - except OSError: - raise OSError('Cython needs to be installed') - - -def process_tempita_pyx(fromfile, tofile): - try: - try: - from Cython import Tempita as tempita - except ImportError: - import tempita - except ImportError: - raise Exception('Building Statsmodels requires Tempita: ' - 'pip install --user Tempita') - with open(fromfile, "r") as f: - tmpl = f.read() - pyxcontent = tempita.sub(tmpl) - assert fromfile.endswith('.pyx.in') - pyxfile = fromfile[:-len('.pyx.in')] + '.pyx' - with open(pyxfile, "w") as f: - f.write(pyxcontent) - process_pyx(pyxfile, tofile) - - -rules = { - # fromext : function - '.pyx' : process_pyx, - '.pyx.in' : process_tempita_pyx - } - - -# -# Hash db -# -def load_hashes(filename): - # Return { filename : (sha1 of input, sha1 of output) } - if os.path.isfile(filename): - hashes = {} - with open(filename, 'r') as f: - for line in f: - filename, inhash, outhash = line.split() - hashes[filename] = (inhash, outhash) - else: - hashes = {} - return hashes - - -def save_hashes(hash_db, filename): - with open(filename, 'w') as f: - for key, value in sorted(hash_db.items()): - f.write("%s %s %s\n" % (key, value[0], value[1])) - - -def sha1_of_file(filename): - h = hashlib.sha1() - with open(filename, "rb") as f: - h.update(f.read()) - return h.hexdigest() - - -# -# Exclusions -# -def load_exclusions(filename): - # Return [ path ] - if os.path.isfile(filename): - with open(filename, 'r') as f: - paths = f.read().splitlines() - else: - paths = [] - return paths - - -# -# Main program -# - -def normpath(path): - path = path.replace(os.sep, '/') - if path.startswith('./'): - path = path[2:] - return path - - -def get_hash(frompath, topath): - from_hash = sha1_of_file(frompath) - to_hash = sha1_of_file(topath) if os.path.exists(topath) else None - return (from_hash, to_hash) - - -def process(path, fromfile, tofile, processor_function, hash_db): - fullfrompath = os.path.join(path, fromfile) - fulltopath = os.path.join(path, tofile) - current_hash = get_hash(fullfrompath, fulltopath) - if current_hash == hash_db.get(normpath(fullfrompath), None): - print('%s has not changed' % fullfrompath) - return - - orig_cwd = os.getcwd() - try: - os.chdir(path) - print('Processing %s' % fullfrompath) - processor_function(fromfile, tofile) - finally: - os.chdir(orig_cwd) - # changed target file, recompute hash - current_hash = get_hash(fullfrompath, fulltopath) - # store hash in db - hash_db[normpath(fullfrompath)] = current_hash - - -def find_process_files(root_dir): - hash_db = load_hashes(HASH_FILE) - exclusions = load_exclusions(EXCLUSION_FILE) - for cur_dir, dirs, files in os.walk(root_dir): - for filename in files: - in_file = os.path.join(cur_dir, filename + ".in") - if filename.endswith('.pyx') and os.path.isfile(in_file): - continue - if os.path.join(cur_dir, filename) in exclusions: - continue - for fromext, function in rules.items(): - if filename.endswith(fromext): - toext = ".c" - with open(os.path.join(cur_dir, filename), 'rb') as f: - data = f.read() - m = re.search(br"^\s*#\s*distutils:\s*language\s*=\s*c\+\+\s*$", data, re.I | re.M) - if m: - toext = ".cxx" - fromfile = filename - tofile = filename[:-len(fromext)] + toext - process(cur_dir, fromfile, tofile, function, hash_db) - save_hashes(hash_db, HASH_FILE) - - -def main(): - try: - root_dir = sys.argv[1] - except IndexError: - root_dir = DEFAULT_ROOT - find_process_files(root_dir) - - -if __name__ == '__main__': - main()