Christoph Deil

Results 98 comments of Christoph Deil

I'm cleaning up the RST docs a bit first before tending to the notebooks. Removed `docs/data/obs_group.rst` in 404ac495 That should be the starting point for this tutorial notebook on obs...

@registerrier - Do you have something here for Monday or Tuesday latest? IMO it's not critical, OK to just postpone to v0.9.

Discussed in the Gammapy weekly call just now. @AtreyeeS - Please create a brief overview of defaults we have now. It could be a brief section after this one: https://docs.gammapy.org/0.14/scripts/index.html#data-reduction-and-datasets...

Initially gammapy.maps was written using mostly Numpy arrays with very little Quantity support. Then MapCoords was added, and some quantity support was added. @adonath - What do you suggest for...

A related question here is whether to use existing data structures (Table and meta dict) directly, or whether to introduce container classes. Numpy and scipy and scikit-image and scikit-learn are...

> I could write more of these stories. @mackaiver - Thanks. This feedback is very helpful. Based on this, I already have a ton of things I want to clean...

> Why not just use an astropy.time.Time? We could. Putting Time and SkyCoord objects in Table, as well as using using QTable was considered in #980 and it was a...

@adonath - See #1186 and #1208 where this was introduced. As mentioned there, the idea is to return and "invalid" sentinel value instead of raising an error. This is what...

The problem is that with `int` there is no good placeholder. If you want to make it a bit more obvious, you could use `-42`.

@adonath - I'd like to release Gammapy v0.8 in a week. What should we do here? OK to defer to v0.9, or do you think this needs to be addressed...