search

gipit / gips

Geospatial Image Processing System
GNU General Public License v2.0
21 stars 13 forks source link

masking via GeoImage.Process() doesn't update image #27

Closed ircwaves closed 9 years ago

ircwaves commented 9 years ago

As described above, no-arg form of Process doesn't update the image. Curiously, if you read the array (GeoImage.Read()) or if you write to another file (GeoImage.Process(otro_archivo)) the masks are appropriately applied.

matthewhanson commented 9 years ago

Is this due to the image not being opened in write mode? It used to be write was the default but now you have to pass it true a the second argument to the GeoImage constructor.

matthewhanson commented 9 years ago

It should throw an error if read only.

matthewhanson commented 9 years ago

As in, it currently doesn't throw an error but it should, not "it should have if opened read only and Process is called"

ircwaves commented 9 years ago

That sounds likely. I didn't know that the default behaviour had changed. I'll see if it works with update=True

ircwaves commented 9 years ago

adding the True took care of it.