Gabe Brammer

Results 60 comments of Gabe Brammer

Thanks @albertfxwang. `filters` should be a list of `eazy.filters.FilterDefinition` objects which have `pivot` as a property, not a method (https://github.com/gbrammer/eazy-py/blob/master/eazy/filters.py#L121). Are you passing something else in `filters`, for example `pysynphot`...

This is implemented in [jwst_utils.match_gwcs_to_sip](https://github.com/gbrammer/grizli/blob/master/grizli/jwst_utils.py#L259), but isn't yet run in the pipeline.

Thanks @jdavies-st. I agree, but the reason to implement things this way in grizli with the SIP headers is so that I can make consistent WCS between HST and JWST...

Agreed. I'll look into the `jwst.tweakreg` version to see if it's better than what I was testing last summer. I'm sorry to admit I generally haven't updated my `jwst` module...

Do you know which keyword has the offending `\n` characters? `COMMENT`, `HISTORY`?

Thanks, @Bren314 . Can you try compiling with the `-e` flag, i.e., `pip install -e . -r requirements.txt` from within the `grizli` repository directory? That should compile the `c` extensions...

I suspect that you were trying to `import grizli` from within the repository directory, which will have problems.

Does `/tmp` exist and do you have write access to it? Dumping things there is hard-coded a few places in grizli and should be done a bit more cleanly. I'm...

Can you say how you compiled and installed the module? Also check that you're not trying to `from grizli import multifit` from within the local directory of the repository itself...

Hi @njcleri. I'm trying to clear out some of the open issues. Do you think this is still a `grizli` issue, or did changing the directory as mentioned above fix...