aaronayres35

many (but potentially not all) links will be fixed by #462

#802 and #804 have been merged. This PR can be closed after we go through and audit the code base to see if anything was missed

So I've gone through the code base and I think it looks like we've got everything. The only thing that stood out was the `map_screen` method on `DataView` which I...

ref: #508

FWIW - This same traceback was seen a while back: https://github.com/enthought/chaco/issues/529 and that issue was closed by https://github.com/enthought/chaco/pull/636 but 636 was just checking for NaN inputs, _not non-NaN inputs that...

As an update: `BaseZoomTool` and `SimpleZoom` were removed in #670 Also, the `ZoomTool` currently exposed in `chaco.tools.api` just links to the `BetterSelectingZoom` class also exposed in the `chaco.tools.api`. `BetterZoom` (also...

Does this require the examples to to moved inside the package or to make `examples` a python package itself with an `__init__.py` file? (this is already planned when we contribute...

Also, the `hittest` method on `BaseXYPlot` returning tuples bothers me a bit

There exist `map_data_array` methods on all the Mapper classes, but they are not used anywhere in the code base. I imagine their original intention was precisely such a distinction, in...

Running list of special handling because of discrepancies on input/return types: - https://github.com/enthought/chaco/blob/8065abbcaeb274407385cf6fff400cc47081d29f/chaco/tools/lasso_selection.py#L323-L327 - https://github.com/enthought/chaco/blob/8065abbcaeb274407385cf6fff400cc47081d29f/chaco/base_xy_plot.py#L443-L446 - https://github.com/enthought/chaco/blob/8065abbcaeb274407385cf6fff400cc47081d29f/chaco/color_bar.py#L156-L157 - https://github.com/enthought/chaco/blob/3259e05a73be3e7ef51a0397ff629a9341907b07/chaco/scatterplot.py#L341-L343 - https://github.com/enthought/chaco/blob/048508394718e88da22c525caae9a3b3e5736a51/chaco/lasso_overlay.py#L69-L70