It would be great if we could launch ipython optionally like:

  ./bin/gmg shell --ipython

There's some code in werkzeug's codebase that looks like it does this the right way.

        if ipython:
            try:
                try:
                    from IPython.frontend.terminal.embed import InteractiveShellEmbed
                    sh = InteractiveShellEmbed(banner1=banner)
                except ImportError:
                    from IPython.Shell import IPShellEmbed
                    sh = IPShellEmbed(banner=banner)
            except ImportError:
                pass
            else:
                sh(global_ns={}, local_ns=namespace)
                return
        from code import interact
        interact(banner, local=namespace)

If we borrow a significant chunk of this code, we should annotate correctly:

​http://www.softwarefreedom.org/resources/2007/gpl-non-gpl-collaboration.html

It used to be that if the image was smaller than the maximum width/height, it would just not create a "medium" image. But when the exif rotation code was added, it was switched so that it would create a medium image no matter what.

Do something along the lines of:

if needs_exif_rotation or greater_than_maxsize:
    medium creating code here

It would be really great if when someone comments on your media entry that you get notified. I know I'd love to know if someone loved my video!

Several parts to this, I think:

• User option to "email notifications of comments"
  • Add to the user interface in the user profile page
  • Add option to the model in Mongo and SQL models (if SQL hasn't landed completely by then!)
  • Add migration to Mongo (or SQL if it's landed before this ;))
• Actually email notify users! See the registration code which has examples of this.
• Possibly turn on and off this option altogether in the config file? See: ​http://wiki.mediagoblin.org/Configure_MediaGoblin (not sure if this is bloat or not ;))
• Unit tests for this would be nice... but then again unit tests of comments at all would be nice. Not a requirement for merge, sadly! ;)