|
From: Federico A. <ari...@gm...> - 2013-09-14 16:12:17
|
Hello Finally (after long time) I managed to get time and courage to make my first PR with this little modification https://github.com/matplotlib/matplotlib/pull/2417 Federico On Tue, Mar 13, 2012 at 3:17 PM, John Hunter <jd...@gm...> wrote: > > > On Tue, Mar 13, 2012 at 1:20 PM, Federico Ariza <ari...@gm...>wrote: > >> Hi >> >> That is exactly what I am doing, but I thought it was kept somewhere. >> >> I like the idea of upstream modification of relim. >> >> > It would be trivial to add a kwarg to relim: > > include_invisible=True > > which defaults to the current behavior. > def relim(self, include_invisible=True): > """ > Recompute the data > limits based on current artists. If you want to exclude > invisible artists from the calculation, set > `include_invisible=False` > > At present, :class:`~matplotlib.collections.Collection` > instances are not supported. > """ > # Collections are deliberately not supported (yet); see > # the TODO note in artists.py. > self.dataLim.ignore(True) > self.ignore_existing_data_limits = True > for line in self.lines: > if include_invisible or line.get_visible(): > self._update_line_limits(line) > > for p in self.patches: > if include_invisible or p.get_visible(): > self._update_patch_limits(p) > > > But include_invisible isn't the most intuitive name... > > JDH > > > > -- Y yo que culpa tengo de que ellas se crean todo lo que yo les digo? -- Antonio Alducin -- |