Skip to content

Commit

Permalink
Bump super-linter/super-linter from 7.1.0 to 7.2.1 (#555)
Browse files Browse the repository at this point in the history
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: sharvil10 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: sharvil10 <[email protected]>
  • Loading branch information
dependabot[bot] and sharvil10 authored Jan 16, 2025
1 parent 0b6a6b0 commit 9662f0d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
with:
fetch-depth: 0
- uses: super-linter/super-linter/slim@b92721f792f381cedc002ecdbb9847a15ece5bb8 # v7.1.0
- uses: super-linter/super-linter/slim@85f7611e0f7b53c8573cca84aa0ed4344f6f6a4d # v7.2.1
env:
GITHUB_ACTIONS_COMMAND_ARGS: '-ignore SC.*'
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -197,4 +197,5 @@ Run the following command on the launcher nodes in a shell to launch the contain
python tensorflow2_keras_mnist.py
> [!NOTE]
> That the number of processes spawned on each node will be dependent on the parameters specified with `horovodrun` command. You can modify the horovodrun command to your needs from the [documentation here](https://horovod.readthedocs.io/en/stable/running_include.html) and [here](https://horovod.readthedocs.io/en/stable/docker_include.html). For the workflow to work all nodes need to communicate with each other and be able to pass the MPI messages among all processes. This requires the correct environment setup based on the specific networking setup you have. [Intel® MPI documentation](https://www.intel.com/content/www/us/en/docs/mpi-library/developer-reference-linux/2021-10/environment-variable-reference.html) provides the necessary environment variables to control the communication between processes using MPI.
6 changes: 3 additions & 3 deletions test-runner/test_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,9 +206,9 @@ def get_test_list(args: dict, tests_yaml: List[dict]):
break
summary.append([idx + 1, test.name, "PASS"])
json_summary.append({"Group": test_group, "Test": test.name, "Status": "PASS"})
json_summary_path = f"{args.logs_path}/{test_group.replace('/', '-')}-summary.json"
JSON_SUMMARY_PATH = f"{args.logs_path}/{test_group.replace('/', '-')}-summary.json"

with open(json_summary_path, "w", encoding="utf-8") as file:
with open(JSON_SUMMARY_PATH, "w", encoding="utf-8") as file:
json.dump(json_summary, file, indent=4)

# Switch logging context back to the initial state
Expand All @@ -223,7 +223,7 @@ def get_test_list(args: dict, tests_yaml: List[dict]):
docker.system.prune()
logging.info("%d Images Removed", len(test_images))
# DEBUG
logging.info("Saved json summary file in: %s", json_summary_path)
logging.info("Saved json summary file in: %s", JSON_SUMMARY_PATH)
# Print Summary Table
logging.info(
"\n%s", tabulate(summary, headers=["#", "Test", "Status"], tablefmt="orgtbl")
Expand Down

0 comments on commit 9662f0d

Please sign in to comment.