diff --git a/pype/lib.py b/pype/lib.py index 449c9f437a..e009ac1940 100644 --- a/pype/lib.py +++ b/pype/lib.py @@ -1663,9 +1663,7 @@ def timeit(method): name = kw.get('log_name', method.__name__.upper()) kw['log_time'][name] = int((te - ts) * 1000) else: - log.debug('%r %2.2f ms' % \ - (method.__name__, (te - ts) * 1000)) - print('%r %2.2f ms' % \ - (method.__name__, (te - ts) * 1000)) + log.debug('%r %2.2f ms' % (method.__name__, (te - ts) * 1000)) + print('%r %2.2f ms' % (method.__name__, (te - ts) * 1000)) return result return timed \ No newline at end of file diff --git a/pype/modules/sync_server/__init__.py b/pype/modules/sync_server/__init__.py index 22a68bed9a..d7590e24e0 100644 --- a/pype/modules/sync_server/__init__.py +++ b/pype/modules/sync_server/__init__.py @@ -2,4 +2,4 @@ from .sync_server import SyncServer def tray_init(tray_widget, main_widget): - return SyncServer() \ No newline at end of file + return SyncServer() diff --git a/pype/modules/sync_server/providers/abstract_provider.py b/pype/modules/sync_server/providers/abstract_provider.py index f0935c22fb..f40a964b2d 100644 --- a/pype/modules/sync_server/providers/abstract_provider.py +++ b/pype/modules/sync_server/providers/abstract_provider.py @@ -8,10 +8,13 @@ class AbstractProvider(metaclass=ABCMeta): """ Copy file from 'source_path' to 'target_path' on provider. Use 'overwrite' boolean to rewrite existing file on provider - :param source_path: absolute path on local system - :param target_path: absolute path on provider (GDrive etc.) - :param overwrite: True if overwite existing - :return: file_id of created file, raises exception + + Args: + source_path (string): absolute path on local system + target_path (string): absolute path on provider (GDrive etc.) + overwrite (boolean): True if overwite existing + Returns: + (string) file_id of created file, raises exception """ pass @@ -19,9 +22,12 @@ class AbstractProvider(metaclass=ABCMeta): def download_file(self, source_path, local_path): """ Download file from provider into local system - :param source_path: absolute path on provider - :param local_path: absolute path on local - :return: + + Args: + source_path (string): absolute path on provider + local_path (string): absolute path on local + Returns: + None """ pass @@ -29,8 +35,12 @@ class AbstractProvider(metaclass=ABCMeta): def delete_file(self, path): """ Deletes file from 'path'. Expects path to specific file. - :param path: absolute path to particular file - :return: None + + Args: + path (string): absolute path to particular file + + Returns: + None """ pass @@ -38,7 +48,10 @@ class AbstractProvider(metaclass=ABCMeta): def list_folder(self, folder_path): """ List all files and subfolders of particular path non-recursively. - :param folder_path: absolut path on provider - :return: + Args: + folder_path (string): absolut path on provider + + Returns: + (list) """ - pass \ No newline at end of file + pass diff --git a/pype/modules/sync_server/sync_server.py b/pype/modules/sync_server/sync_server.py index 6f2486884d..7aac5952e8 100644 --- a/pype/modules/sync_server/sync_server.py +++ b/pype/modules/sync_server/sync_server.py @@ -114,7 +114,8 @@ class SyncServer(): Returns: (list) """ - retries_str = "null,"+",".join([str(i) for i in range(self.RETRY_CNT)]) + retries_str = "null," + \ + ",".join([str(i) for i in range(self.RETRY_CNT)]) representations = io.find({ "type": "representation", "$or": [ @@ -399,7 +400,7 @@ class SyncServer(): "files._id": file_id } update = { - "$unset": {"files.$.sites.{}".format(site_index): ""} + "$unset": {"files.$.sites.{}".format(site_index): ""} } # it actually modifies single _id, but io.update_one not implemented io.update_many( @@ -533,7 +534,6 @@ class SynchServerThread(threading.Thread): try: while self.is_running: import time - from datetime import datetime start_time = time.time() sync_representations = self.module.get_sync_representations()