search

astrofrog / pyavm

Pure-python AVM library
http://astrofrog.github.io/pyavm/
Other
20 stars 9 forks source link

Minor fix to Controlled Vocabulary #21

Closed keflavich closed 10 years ago

keflavich commented 10 years ago

At least, I think it's a fix. I couldn't read the AVM from this Chandra image: http://chandra.harvard.edu/photo/2009/gcenter/

so I added Log(x) to the STRETCH_FUNCTION_CHOICES.

astrofrog commented 10 years ago

@keflavich - thanks! Could you double check what the standard defines? If they are not strictly the standard, I wonder whether it would be worth adding a pedantic mode which does not accept these but is off by default?

astrofrog commented 10 years ago

In any case this should probably not crash, just warn.

keflavich commented 10 years ago

I can't find anything about the STRETCH_FUNCTION in the AVM standard. I'll look further, but no guarantees I'll find anything

astrofrog commented 10 years ago

I fixed it with 6178d81259355279980b82b61da7978fb0b2b593 instead