2013 10 01

#amara

/

      • msaad has left the channel
      • unisub-jenkins-b
        Project unisubs-staging build #2104: STILL FAILING in 5 hr 38 min: http://dev.universalsubtitles.org:8888/job/unis...
      • * ben: Fixed UniSubURLValidator to load the Site object lazily
      • * ben: Added model class that tracks videos moving from team to team
      • * ben: Fixed attribute error in the billing test
      • * ben: BillingReport refactoring
      • * ben: Couple of speedups for the unittests
      • * ben: Implemented billing report for approval-type teams
      • * ben: Made approval billing reports not select rejected approval tasks
      • * ben: Remove subtitler/reviewer replace with approver for billing reports
      • * ben: Added Translation? column to the approval billing reports.
      • * ben: Renamed billing record types and removed the old type
      • * ben: Added billing type to pay users who perform subtitling/review work
      • * ben: Fixed attribute error with the billing form
      • * ben: Handle running approval for users tasks without review enabled (#890)
      • * ben: Encode as utf-8 user names before writing them to the CSV file (#890)
      • * janet dragojevic: Update tests for billing changes.
      • * Evan: refs 890: fixed unicode error for body
      • * janet dragojevic: update selenium test for unicode notes
      • maggie_s_
        all: out for a bit, back in a couple of hours
      • nicholasreville joined the channel
      • jenny__ has quit
      • janeted
        ehazlett: can we deploy production?
      • oh - you did.
      • thanks
      • nicholasreville has quit
      • ehazlett: looks like unilangs needs updating on production, I don't see the new language added.
      • Sithu joined the channel
      • Sithu
        Hello everyone!
      • What is Unisub?
      • Sithu has left the channel
      • franontanaya joined the channel
      • franontanaya is now known as Guest25795
      • janeted1 joined the channel
      • janeted has quit
      • janeted1 has quit
      • janeted joined the channel
      • Guest25795 is now known as franontanaya
      • maggie_s_ has quit
      • nicholasreville joined the channel
      • janeted
        all: heading out for a bit to take the kids to swimming.
      • ehazlett: can you please update unilangs on production.
      • nicholasreville has quit
      • janeted has quit
      • bendk has quit
      • michelled joined the channel
      • ehazlett
        all: unilangs should be updated and production re-deployed
      • janeted joined the channel
      • janeted1 joined the channel
      • janeted has quit
      • janeted1 has quit
      • janeted joined the channel
      • nigel_bot has quit
      • nigel_bot joined the channel
      • janeted
        hi ehazlett can you check that ted feed import celery task again. I think there may be new videos on their feed again.
      • ehazlett
        janeted: yeah i need to check celery i think
      • it looks like the celerybeat [get_bloginfo]url[/get_bloginfo]/files/freenode/amara/isn't running or something
      • janeted
        thanks
      • bendk joined the channel
      • bendk has quit
      • bendk joined the channel
      • hi bendk welcome back
      • bendk
        hey janet
      • almost done with the billing report stuff
      • so I'm working on issue #3 and I have a question
      • is it normal to have an approved task without a subtitle/translate task?
      • or any other task for that matter
      • that's the situation happening and I'm wondering if it should be an error, or if we should just ignore it
      • janeted
        bendk: that shouldn't be possible - unless it was manually created
      • bendk
        yeah, I don't know what to think about it
      • video_id=aZcgvT4n4QEA
      • language =ja
      • Maybe we should add a row that says "no approver"
      • then terri will see that something is up
      • janeted
        I think we should ignore it, since there's nothing to stop an admin from manually creating a task. It could be a case where they turned on tasks after it was subtitled
      • bendk
        okay, ignoreing is fine with me as well
      • janeted
        so I'm hoping it's just an anomaly of the getting things started in their process.
      • another case would be importing subs from another team
      • importing videos, i mena
      • how did you solve the time thing?
      • bendk
        I just found the first time that we could use
      • janeted
        aha - was it caused by subs without an end time on the last sub?
      • bendk
        so for the start time, go through the subtitles from start to end, if the subtitle has a start time use that, if the subtitle has an end time use that, otherwise move to the next one
      • and do the reverse for end times
      • janeted: yeah
      • janeted
        makes sense - I was guess that
      • bendk
        not sure how that happened either since then the subtitles shouldn't be complete
      • but it's not too hard to work around
      • janeted
        we got a slew of vidoes from udacity that were marked as complete but the last sub didn't have an end time
      • it wreaked havoc on the tasks stuff.
      • bendk: indent error
      • File "apps/teams/search_indexes.py", line 28, in <module>
      • from teams import models
      • File "apps/teams/models.py", line 2633
      • try:
      • I have to drive home now, be back in 1 hr to finish testing
      • janeted has quit
      • janeted joined the channel
      • hi bendk
      • bendk
        hey janeted
      • janeted
        just ran the tests and got
      • ERROR 2013-10-01 13:08:32,808 tasks 5523 140566216734464 Celery job exception: TypeError('tuple' object does not support item assignment)
      • Traceback (most recent call last):
      • File "/opt/ve/vagrant/unisubs/lib/python2.6/site-packages/celery/task/trace.py", line 224, in trace_task
      • R = retval = fun(*args, **kwargs)
      • File "apps/teams/tasks.py", line 187, in process_billing_report
      • report.process()
      • File "/usr/lib/python2.6/contextlib.py", line 34, in __exit__
      • self.gen.throw(type, value, traceback)
      • File "/opt/apps/vagrant/unisubs/utils/metrics.py", line 135, in Timer
      • yield
      • File "apps/teams/tasks.py", line 187, in process_billing_report
      • report.process()
      • File "apps/teams/models.py", line 2690, in process
      • in convert_unicode_to_utf8
      • TypeError: 'tuple' object does not support item assignment
      • bendk
        janeted: ahh, I can fix that
      • 1 sec
      • janeted: should be fixed
      • michelled has quit
      • janeted
        bendk: all tests pass now, thanks
      • testing the other one now, but I'm not sure how to make it fail, so I guess will just make sure it all still works
      • bendk
        janeted: yeah, there shouldn't be a good way to force it
      • ehazlett: can you take a look at that pull request ^^
      • ehazlett
        bendk: sure
      • bendk
        ehazlett: also, I'm wondering why the TED feed import [get_bloginfo]url[/get_bloginfo]/files/freenode/amara/isn't working on production
      • is it possible that the worker container doesn't have the integration repo?
      • ehazlett
        bendk: i'm looking into celery
      • bendk
        that's my guess as to what's happening
      • ehazlett
        no -- the container is the same for all
      • bendk
        because it works when you run it manually
      • hmm
      • ehazlett
        there is only one container
      • bendk
        I think it should run every hour
      • ehazlett
        er image
      • bendk
        so there should be logs
      • ehazlett
        yeah
      • bendk
        okay, I'll just let you handle it
      • ehazlett
        i think celerybeat is not working
      • k
      • bendk
        tell me if you run into anything weird
      • ehazlett
        yeah i'm looking into it
      • thx will do
      • bendk
        yeah, celerybeat not working would definitely do it
      • janeted
        ehazlett: here's the pull request for the other billing one
      • ehazlett
        k
      • janeted
        bendk: ehazlett can we get these right up on production Aleli really needs these reports