Skip to content
Snippets Groups Projects

WIP: Add a bob em train script which works on SGE

Closed WIP: Add a bob em train script which works on SGE
bob_em_train into master
1 unresolved thread
Closed Amir MOHAMMADI requested to merge bob_em_train into master
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
bob/learn/em/script/train.py 0 → 100644
2 """
3 import logging
4 import click
5 import subprocess
6 import os
7 import sys
8 import shutil
9 import time
10 import numpy as np
11 from bob.extension.scripts.click_helper import (
12 verbosity_option,
13 ConfigCommand,
14 ResourceOption,
15 log_parameters,
16 )
17 from bob.extension.processors import vstack_features
  • Amir MOHAMMADI added 1 commit

    added 1 commit

    • c898cbf3 - Fix the import for the moved function

    Compare with previous version

  • Amir MOHAMMADI added 1 commit

    added 1 commit

    • 093c7c62 - Improvements to the em train script

    Compare with previous version

  • Amir MOHAMMADI
    Amir MOHAMMADI @amohammadi ·
    Author Owner

    This is shaping up nicely. I would love to get some feedback on this from Bob developers.

  • Amir MOHAMMADI added 1 commit

    added 1 commit

    • 6e804b8a - Improvements to the em train script

    Compare with previous version

  • Tiago de Freitas Pereira
    Tiago de Freitas Pereira @tiago.pereira ·
    Owner

    Hey man,

    Thanks for the feature.

    I will look in to this + bob.measure!95 (merged) + bob.bio.base#106 (closed) when I come back from holidays. Yes, I have a lot of extra stuff to look at :-P

  • Amir MOHAMMADI added 1 commit

    added 1 commit

    Compare with previous version

  • Amir MOHAMMADI
    Amir MOHAMMADI @amohammadi ·
    Author Owner

    Now that we know what dask is :) maybe we should implement this MR based on dask.

  • Tiago de Freitas Pereira
    Tiago de Freitas Pereira @tiago.pereira ·
    Owner

    Yes, we could. I will close this one and open another one in the future that handles that.

    For the record, this is the reference implementation https://github.com/tiagofrepereira2012/tam-dask

  • Tiago de Freitas Pereira closed

    closed

  • Amir MOHAMMADI
    Amir MOHAMMADI @amohammadi ·
    Author Owner

    If you want to work on this, you can just push into this branch. I don't use this code.

    • Please register or sign in to reply