A few minor changes in response to the copilot review of astropy#191

Author: mwcraig

astrowidgets/bqplot.py

@@ -114,7 +114,7 @@ def data_aspect_ratio(self):
         """
         Aspect ratio of the image data, horizontal size over vertical size.
         """
-        return self._image_shape[0] / self._image_shape[1]
+        return self._image_shape[1] / self._image_shape[0]
 
     def reset_scale_to_fit_image(self):
         wide = self.data_aspect_ratio < 1
@@ -467,7 +467,7 @@ def _add_new_single_marker(self, x_mark, y_mark):
                                           marker_name,
                                           color=self.marker['color'],
                                           size=self.marker['radius']**2,
-                                          style=self.marker['type'])
+                                          shape=self.marker['type'])
 
     # Update the viewport to match changes in the UI
     def _init_watch_image_changes(self):
@@ -499,8 +499,8 @@ def update_zoom_level(event):
         # or the minimum instead of the maximum is also fine.
         x_scale = self._astro_im._scales['x']
 
-        # THIS IS TERRIBLE AND MAKES THINGS SUPER LAGGY!!!! Needs to be
-        # throttled or something. Look at the ImageGL observe options.
+        # If things seem laggy in the future, check whether throtttling
+        # the updates helps.
         x_scale.observe(update_zoom_level, names='max')
 
     def _interval_and_stretch(self, stretch=None, cuts=None):