search

populse / mia_processes

The default processes repository for mia
Other
1 stars 2 forks source link

Update ASL pipeline #2

Closed BlancaNietoPetinal closed 4 years ago

servoz commented 5 years ago

Hola Blanca, You are the first to contribute to the mia_processes repository ... Before, I was the only contributor in this repository. I must therefore define a general guideline for the review of the pull request! So, please be patient, I will take the time (not too much, I promise!) to see how to proceed (exact definition of the rules for the acceptance of future pull requests; docstring, documentation, code guide lines, etc...)

servoz commented 4 years ago

I guess this PR is now obsolete, so I close it. I had indicated in the first post that I would propose guide lines for the mia_processes library. To this end, I propose that we use as an example the bricks and associated doc in the current mia_processes V1.4.0 (pypi). This example could evolve later, according to the proposals of the team's developers (there is already an interesting proposal to manage the doc, by denisri #4 and #3). However we will have to be strict enough in the respect of the example choosen in order to have a library which is not a mess and which is perfectly functional.