Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation from source fails on centos 7 with gcc 4.8.5 #26

Open
tagordon opened this issue Jan 28, 2021 · 4 comments
Open

Installation from source fails on centos 7 with gcc 4.8.5 #26

tagordon opened this issue Jan 28, 2021 · 4 comments
Labels
bug Something isn't working

Comments

@tagordon
Copy link
Collaborator

Hi,

I'm trying to install from source following the directions here. The reason I'm installing from source rather than the pre-compiled binaries is that I'm hoping to try out the Kronecker functionality. It works fine on my mac, but when I try to install on the cluster I get the error:

ERROR: Command errored out with exit status -11: /gscratch/home/tagordon/jwst/celerite2/env/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/gscratch/home/tagordon/jwst/celerite2/setup.py'"'"'; file='"'"'/gscratch/home/tagordon/jwst/celerite2/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' develop --no-deps Check the logs for full command output.

This happens regardless of whether I'm install from the kron branch or not. Any help is appreciated!

Thanks,
Tyler

@dfm
Copy link
Member

dfm commented Jan 28, 2021

I think this is the same issue as #3 and the problem is that the compilation uses an unreasonable amount of memory because of how it's handling fixed size matrices.

A temporary fix would probably be to replace lines 27-107 in python/celerite2/driver.hpp with something like:

#define UNWRAP_CASES_FEW                     \
  switch (J) {                               \
    case 1: FIXED_SIZE_MAP(1); break;        \
    case 2: FIXED_SIZE_MAP(2); break;        \
    default: FIXED_SIZE_MAP(Eigen::Dynamic); \
  }

#define UNWRAP_CASES UNWRAP_CASES_FEW
#define UNWRAP_CASES_MOST UNWRAP_CASES_FEW

@dfm
Copy link
Member

dfm commented Jan 28, 2021

I have plans for better fixes but I have plans for lots of things…

@dfm dfm added the bug Something isn't working label Jan 28, 2021
@tagordon
Copy link
Collaborator Author

So, making that substitution didn't seem to change anything but I was able to install it by increasing my memory allocation on the cluster, which must mean that you diagnosed the problem correctly. Thanks!

@dfm
Copy link
Member

dfm commented Jan 28, 2021

Bizarre that that isn't the source of the problem though… Thanks for the update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants