Commit 9e3ad890 authored by Jerome Mariette's avatar Jerome Mariette
Browse files

answer to reviewer 2 ok

parent 0e319928
......@@ -97,7 +97,9 @@ position or a flag for each parameter. Jflow also embeds an XML parser which
allows to run genuine Mobyle (Wilkinson et al., 2009) components. Finally, to
allow developers to integrate components from other WMS, Jflow provides a
skeleton class. This class only requires to implement the parsing step. A
workflow chains several components. A Jflow workflow is built as a
workflow chains components. It is represented by a directed acyclic graph (DAG)
where nodes represent jobs and edges links between inputs and outputs. When
paths are disjoint, jobs are run in parallel. A Jflow workflow is built as a
\textit{Workflow} subclass. Components are added to the workflow as variables
and chained linking outputs and inputs.
......@@ -113,7 +115,7 @@ weaver (Bui et al., 2012). Therefore Jflow manages error recovery and supports
most distributed resource management systems (Condor, SGE, Work Queue or a
single multi core machine, \ldots). Replacing Makeflow by an other job submitter
requires to implement a new \textit{Engine} subclass. This class creates and
executes the workflow directed acyclic graph (DAG).
executes the workflow DAG.
\section{Example}
......
......@@ -95,6 +95,11 @@ graphs. The figure shows that this is the case for Jflow as well. I think, a few
sentences about what theoretical workflow model is supported by Jflow and its
server component should be added.}
The following sentence has been added to the text ``A workflow chains
components. It is represented by a directed acyclic graph (DAG) where nodes
represent jobs and edges links between inputs and outputs. When paths are
disjoint, jobs are run in parallel.''
\textbf{- The references should list all authors, not only the first.}
All authors of the references have been added.
......@@ -106,7 +111,8 @@ acceptable for newly released software. Also, I would like to propose to package
and distribute Jflow properly as a Pypi or even a Conda package to simplify
installation and upgrades.}
Jflow has been rewritten to be Python3 complient.
Jflow has been rewritten to be Python3 complient. Distributing jflow as a Pypi package
will be considered for the next release of Jflow.
\textbf{Minor comments:}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment