So far, the auto steps include:

1. after a PR is merged, zip all bundles and check which one has been changed.
2. for changed bundles, upload the new zip file to https://github.com/Project-MONAI/model-zoo/releases/tag/hosting_storage_v1
3. update new version, checksum and download into models/model_info.json, and push into a new branch, the branch name is in the form of [merged PR number]-auto-update-model-info.

Then, an admin/contributor is able to create a new pull request of this created new branch (this create pr is still manual, I tried to add gh cli within update_model_info.py, but met the error like here, Hi @wyli , do you have any suggests about how to automatically make the PR? )

Another place that to be added is the uploading results check, since the upload thing may fail due to network or other issues, I will address this part later.

looks like a permission issue you can try

name: Push to master

on:
  push:
    branches:
      - master

jobs:
  build:
    runs-on: ubuntu-latest
    steps:
    # the checkout action persists the passed credentials by default
    # subsequent git commands will pick them up automatically
    - uses: actions/checkout@v2
      with:
        token: ${{secrets.PAT}}
    - run: |
        # do something
        git push

https://stackoverflow.com/a/64078507

Thanks @wyli , the push part is fine, and the issue comes from the auto pull request part. I used gh pr create ..., but it seems there are some prompts after the command.

Hi @Nic-Ma
FYI: with the commit (link), admin can do:

1. download large files.
2. compress (exclude the large files config file) the bundle to a zip file
3. upload the zip file into releases
   by running the following command:

python .github/update_model_info.py -b bundle_name

The next step is to put it into a workflow.

Hi @yiheng-wang-nv ,

Great improvement, please hold on for the workflow, maybe let's put it into Blossom directly.
CC @YanxuanLiu .

Thanks.

I see you commented much code in this PR, those parts are not ready to use?

Thanks.

Hi @wyli @Nic-Ma and @ericspod , I just marked the status of this PR into "Ready", and could you please help to review it?
I attached a flow chart at the beginning, and it might be helpful to understand the processes.

During a previous commit, I used the spleen ct segmentation bundle for test: log, uploaded place, produced new branch

thanks, this looks great, does this support deletion as well? just wanted to confirm that if I submit a PR to permanently remove a bundle folder, the PR won't break the pipeline. And I guess we should provide some instructions to the users/admins about how to remove a bundle folder.

Hi @wyli ,

That's a good corner case, @yiheng-wang-nv please make sure a PR can delete some bundles in the source code(not in the release storage).

Thanks.

The version strings are part of the compressed file's names so we should mention in the specification that versions should follow semantic versioning and contain only characters valid in filenames.

thanks, this looks great, does this support deletion as well? just wanted to confirm that if I submit a PR to permanently remove a bundle folder, the PR won't break the pipeline. And I guess we should provide some instructions to the users/admins about how to remove a bundle folder.

Thanks! @wyli and @Nic-Ma . If delete a whole bundle, according to the logic (utils.get_changed_bundle_list, it is used to grab all changed and exsiting bundles), the github release part will not be changed actually. I just pushed a new commit to print some information to address this case.

The version strings are part of the compressed file's names so we should mention in the specification that versions should follow semantic versioning and contain only characters valid in filenames.

Hi @ericspod ,

I created a small PR for it: Project-MONAI/MONAI#4512, could you please help review it?
And I think this PR is ready to merge now.

Thanks in advance.