Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coverity fixes #921

Closed
wants to merge 497 commits into from
Closed

Coverity fixes #921

wants to merge 497 commits into from

Conversation

Garra1980
Copy link
Contributor

Please review these guidelines to help with the review process:

  • Have you provided a meaningful PR description?
  • Have you added a test, a reproducer, or a reference to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?
  • Have you organized your commits logically and ensured each can be built by itself?

pbchekin and others added 30 commits January 5, 2024 13:20
This is flagged as error by hadolint.
…sable workflows (#310)

* Fixed Nightly CI workflow

Signed-off-by: Gregory Shimansky <[email protected]>

* Added secrets inherit when calling reusable workflows

Signed-off-by: Gregory Shimansky <[email protected]>

* Added missing rependency for sending report

Signed-off-by: Gregory Shimansky <[email protected]>

* Fixed conditional operator

Signed-off-by: Gregory Shimansky <[email protected]>

* Fixed runs-on for generate_report

Signed-off-by: Gregory Shimansky <[email protected]>

* Removed report because it cannot be revived

Signed-off-by: Gregory Shimansky <[email protected]>

* Execute cpu and gpu builds in parallel

Signed-off-by: Gregory Shimansky <[email protected]>

---------

Signed-off-by: Gregory Shimansky <[email protected]>
…kflow to build it (#347)

* Initial Dockerfile for FS simulator installation

Signed-off-by: Gregory Shimansky <[email protected]>

* Updated FS simulator Dockerfile to allow simulator setup functions to work

Signed-off-by: Gregory Shimansky <[email protected]>

* Added workflow to build FS simulator image for github actions runners

Signed-off-by: Gregory Shimansky <[email protected]>

---------

Signed-off-by: Gregory Shimansky <[email protected]>
* Fixed Dockerfile after update to KMD archive

Signed-off-by: Gregory Shimansky <[email protected]>

* Added functionality to run FS simulator, excluded long running tests

Signed-off-by: Gregory Shimansky <[email protected]>

* Added workflow to run GPU tests on FS simulator

Signed-off-by: Gregory Shimansky <[email protected]>

* Removed from run on PR, added installation of psutil

Signed-off-by: Gregory Shimansky <[email protected]>

---------

Signed-off-by: Gregory Shimansky <[email protected]>
* Moved installation of psutil into docker image

Signed-off-by: Gregory Shimansky <[email protected]>

* Updated FS sim workflow to use tests exit code in the end

Signed-off-by: Gregory Shimansky <[email protected]>

* Removed pipefail to allow script to finish if tests fail

Signed-off-by: Gregory Shimansky <[email protected]>

* Added upload of simulator individual logs

Signed-off-by: Gregory Shimansky <[email protected]>

* Increase test timeout to 20 minutes

Signed-off-by: Gregory Shimansky <[email protected]>

* Changed timeout to 10 minutes. Added 4 tests to sim exclude.

Signed-off-by: Gregory Shimansky <[email protected]>

* Added two more tests to exclude on FS sim

Signed-off-by: Gregory Shimansky <[email protected]>

---------

Signed-off-by: Gregory Shimansky <[email protected]>
This reverts commit cf80bbf.
leshikus and others added 28 commits September 17, 2024 21:39
separate out load and prefetch configs
* Add --sheet_name and --runtime options

* Add --code_version option, rename default folder names, modify README

* Ignore commented lines while reading csv

* Run the tests as the same order specified in the `input_shapes.csv`

* Add --test_dir option for non-default `generated_gemm`

* Add --timeout to abort long running test cases.

* Parameterize --timeout

* Move the array length check inside `if [[ "$VALIDATE" == 0 ]]`.

* Update README and other small fix.
@Garra1980 Garra1980 closed this Oct 10, 2024
@Garra1980 Garra1980 deleted the coverity_fixes branch October 10, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.