Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TODOs in the code #121

Open
17 of 21 tasks
cdeil opened this issue Apr 7, 2013 · 2 comments
Open
17 of 21 tasks

TODOs in the code #121

cdeil opened this issue Apr 7, 2013 · 2 comments
Milestone

Comments

@cdeil
Copy link
Contributor

cdeil commented Apr 7, 2013

@pwaller Can you check which of these and the +1 year old issues is still relevant and which have been resolved by now?

$ egrep -n -i 'pwaller' weboot/*.py weboot/*/*.py weboot/*/*/*.py  
  • weboot/resources/actions.py:146: (pwaller) doesn't currently know how to achieve this reliably.
  • weboot/resources/actions.py:232: TODO(pwaller): Support for classes, link to online viewer
  • weboot/resources/combination.py:264: # TODO(pwaller): bring back draw options
  • weboot/resources/combination.py:449: # TODO(pwaller): bring back draw options
  • weboot/resources/filesystem.py:70: # TODO(pwaller): This belongs inside the RootFileTraverser
  • weboot/resources/locationaware.py:30: TODO(pwaller): Fix or eliminate this
  • weboot/resources/renderable.py:173: # TODO(pwaller): Remove defunct raise/parentage
  • weboot/resources/renderable.py:196: # TODO(pwaller): special case
  • weboot/resources/renderable.py:209: # TODO(pwaller): figure out why these two lines are preventing
  • weboot/resources/renderable.py:211: # Response(pwaller): global canvas lock seems to have fixed the
  • weboot/views/multitraverse.py:108: # TODO(pwaller): If a context doesn't have an icon, we should
  • weboot/resources/root/graph.py:15: # TODO(pwaller): Introduce some options here..
  • weboot/resources/root/histogram.py:22: TODO(pwaller): Fixme
  • weboot/resources/root/histogram.py:109: # TODO(pwaller): bring back draw options
  • weboot/resources/root/histogram.py:375: # TODO(pwaller): use resource string
  • weboot/resources/root/tree.py:56: # TODO(pwaller): gDirectory needs to be thread-unique. Otherwise:
  • weboot/resources/root/tree.py:58: # TODO(pwaller): Parse self.binning, call appropriate h.

Not yet delt with:

  • weboot/resources/root/tree.py:72: # BUG: TODO(pwaller): Memory leak
  • weboot/resources/multitraverser.py:426: # TODO(pwaller): This is very inefficient and causes OOM quite quickly..
  • weboot/resources/renderable.py:145: # TODO(pwaller): Investigate removing this kludge now we have
  • weboot/resources/root/histogram.py:327: # BUG TODO(pwaller): Not leak memory
@ghost ghost assigned pwaller Apr 7, 2013
@pwaller
Copy link
Member

pwaller commented Apr 7, 2013

Updated issue to use checkboxes

@pwaller
Copy link
Member

pwaller commented Apr 7, 2013

I've done what I'm going to do for today. Will revisit these other things again in the future.

@pwaller pwaller removed their assignment Apr 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants