activitysim icon indicating copy to clipboard operation
activitysim copied to clipboard

Agreed upon and publish process for working with large investments outside the partnership

Open bstabler opened this issue 4 years ago • 3 comments
trafficstars

As discussed at the 09/16/21 meeting, the draft process and phases for working with large investments outside the partnership is:

  • Before work

    • create issue assigned to agency owner and then consortium decides to add the issue to the project board as in progress

      • this is a soft approval to continue with the work with the idea that the contribution will be incorporated
      • our first real test of this process is the SANDAG cross border example
      • agency shares/documents why this work is valuable, for example:
        • adds functionality for special market models
        • the cross border model effort proves asim is extensible
        • the cross border model runtimes are 50% faster than the old version
    • the expectation is that the agency owner will share progress and discuss the work with the consortium as the work progresses

    • A bench reviewer may be decided upon at this point in the process and it should not be someone from the same organization as the author

  • During work

    • add notes / discussion to the issue
  • Submit work

    • create PR
  • Review work / approval

    • Reviewed by other bench contractor
    • This is what Task 5 budgets
    • Revisions as needed
    • Pull (or accept) the PR

Bug fixes / small stuff will be outside this formal process. We decided to try this out for the SANDAG cross border model improvements and then finalize this plan with the partners. @wusun2 will provide an overview on the next call and @danielsclint will do a mini-review to provide feedback.

bstabler avatar Sep 17 '21 00:09 bstabler

see #484

bstabler avatar Sep 22 '21 21:09 bstabler

We agreed on the 10/7 call that @joecastiglione will work on getting a process/governance docs update team together to formalize/memorialize this process.

bstabler avatar Oct 07 '21 17:10 bstabler

see also #496

bstabler avatar Oct 27 '21 17:10 bstabler