#0: Workflow to test compatibility with TTNN Project Template #14442

afuller-TT · 2024-10-29T20:43:57Z

Ticket

None

Problem description

Making changes to CMake. Don't want to manually audit compatibility. Now it's a click away.

What's changed

New run-it-yourself workflow. It can pass (after tenstorrent/cpp-ttnn-project-template#3 goes it to fix it): https://github.com/tenstorrent/tt-metal/actions/runs/11605407955/job/32315761838

Checklist

Post commit CI passes
Blackhole Post commit (if applicable)
Model regression CI testing passes (if applicable)
Device performance regression CI testing passes (if applicable)
New/Existing tests provide coverage for changes

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (1/8)

...tt_metal/tt_metal/unit_tests_fast_dispatch/multichip/test_eth_ring_gather_EnqueueProgram.cpp

ttnn/cpp/ttnn/operations/ccl/reduce_scatter/device/reduce_scatter_op.cpp

ttnn/cpp/ttnn/operations/bernoulli/device/bernoulli_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_layer_norm/device/moreh_layer_norm_device_operation.hpp

ttnn/cpp/ttnn/operations/eltwise/binary/device/broadcast_height_multi_core_program_factory.cpp

ttnn/cpp/ttnn/operations/moreh/moreh_norm/device/moreh_norm_device_operation.hpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (2/8)

ttnn/cpp/ttnn/operations/moreh/moreh_arange/device/moreh_arange_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_matmul/device/moreh_matmul_device_operation.hpp

...lization/layernorm_distributed/device/multi_core/layernorm_post_all_gather_op_multi_core.cpp

ttnn/cpp/ttnn/operations/conv/conv2d/device/conv2d_op_sharded_program_factory.cpp

...mental/transformer/rotary_embedding_llama/device/rotary_embedding_llama_device_operation.cpp

ttnn/cpp/ttnn/operations/normalization/layernorm/device/multi_core/layernorm_op_multi_core.cpp

tests/ttnn/unit_tests/gtests/ttnn_multi_command_queue_fixture.hpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (3/8)

...ions/eltwise/binary/device/broadcast_height_multi_core_sharded_optimized_program_factory.cpp

tests/tt_metal/tt_metal/unit_tests_common/common/common_fixture.hpp

ttnn/cpp/ttnn/operations/normalization/softmax/device/multi_core/softmax_op_multi_core.cpp

ttnn/cpp/ttnn/operations/moreh/moreh_getitem/device/moreh_getitem_rm_factory.cpp

...l/tt_metal/unit_tests_fast_dispatch_single_chip_multi_queue/common/command_queue_fixture.hpp

tests/tt_metal/tt_metal/unit_tests_fast_dispatch/multichip/test_eth_EnqueueProgram.cpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (4/8)

tests/tt_metal/tt_metal/unit_tests/multichip/basic_eth_kernels.cpp

ttnn/cpp/ttnn/operations/data_movement/clone/device/clone_device_operation.hpp

...up_norm_backward/device/input_grad/moreh_group_norm_backward_input_grad_device_operation.hpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (5/8)

ttnn/cpp/ttnn/operations/moreh/moreh_cumsum/device/moreh_cumsum_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_softmax/device/moreh_softmax_device_operation.hpp

ttnn/cpp/ttnn/operations/full/device/full_device_operation.hpp

...alization/layernorm_distributed/device/multi_core/layernorm_pre_all_gather_op_multi_core.cpp

.../cpp/ttnn/operations/moreh/moreh_sum_backward/device/moreh_sum_backward_device_operation.hpp

tests/tt_metal/tt_metal/unit_tests/multichip/erisc_app_direct_send.cpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (6/8)

ttnn/cpp/ttnn/operations/moreh/moreh_group_norm/device/moreh_group_norm_device_operation.hpp

...pp/ttnn/operations/moreh/moreh_norm_backward/device/moreh_norm_backward_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_getitem/device/moreh_getitem_tilized_factory.cpp

ttnn/cpp/ttnn/operations/normalization/groupnorm/device/multi_core/groupnorm_op_multi_core.cpp

...ations/experimental/transformer/rotary_embedding/device/rotary_embedding_program_factory.cpp

...h/moreh_layer_norm_backward/device/moreh_layer_norm_backward_input_grad_device_operation.hpp

tests/tt_metal/tt_metal/unit_tests_common/common/common_fixture.hpp

...imental/transformer/rotary_embedding_llama/device/rotary_embedding_llama_program_factory.cpp

ttnn/cpp/ttnn/operations/eltwise/binary/device/broadcast_width_multi_core_program_factory.cpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (7/8)

tests/tt_metal/tt_metal/unit_tests/common/core_coord_fixture.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_sgd/device/moreh_sgd_device_operation.hpp

tests/tt_metal/tt_metal/unit_tests_common/common/common_fixture.hpp

ttnn/cpp/ttnn/operations/experimental/ccl/all_reduce/device/all_reduce_op.cpp

...eh_layer_norm_backward/device/moreh_layer_norm_backward_gamma_beta_grad_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_sum/device/moreh_sum_device_operation.hpp

...n/operations/moreh/moreh_softmax_backward/device/moreh_softmax_backward_device_operation.hpp

ttnn/cpp/ttnn/operations/moreh/moreh_dot/device/moreh_dot_device_operation.hpp

...n/operations/eltwise/binary/device/broadcast_height_and_width_multi_core_program_factory.cpp

github-actions

⚠️ Clang-Tidy found issue(s) with the introduced code (8/8)

ttnn/cpp/ttnn/operations/uniform/device/uniform_device_operation.hpp

tt-rkim · 2024-10-31T18:21:16Z

.github/workflows/cpp-ttnn-project.yaml

+
+      # TTNN project
+      - name: Checkout cpp-ttnn-project-template
+        uses: actions/checkout@v3


Any reason to use v3?

…rrent#14442)

afuller-TT temporarily deployed to dev October 29, 2024 21:02 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 21:05 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 22:00 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 22:45 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 22:51 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 22:52 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 22:54 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 23:01 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 29, 2024 23:07 — with GitHub Actions Inactive

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from 67a702a to 859d390 Compare October 29, 2024 23:26

afuller-TT temporarily deployed to dev October 29, 2024 23:26 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 30, 2024 02:10 — with GitHub Actions Inactive

afuller-TT temporarily deployed to dev October 30, 2024 16:50 — with GitHub Actions Inactive

github-actions bot reviewed Oct 30, 2024

View reviewed changes

ttnn/cpp/ttnn/operations/uniform/device/uniform_device_operation.hpp Outdated Show resolved Hide resolved

afuller-TT temporarily deployed to dev October 30, 2024 18:41 — with GitHub Actions Inactive

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from 35371ee to 144f32e Compare October 30, 2024 19:18

afuller-TT temporarily deployed to dev October 30, 2024 19:18 — with GitHub Actions Inactive

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from 144f32e to 7a240c9 Compare October 30, 2024 20:28

afuller-TT temporarily deployed to dev October 30, 2024 20:28 — with GitHub Actions Inactive

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from 7a240c9 to b958940 Compare October 30, 2024 20:29

afuller-TT temporarily deployed to dev October 30, 2024 20:29 — with GitHub Actions Inactive

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch 2 times, most recently from 171b3cb to 3318c40 Compare October 30, 2024 22:20

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch 5 times, most recently from 5ba8c8e to 4a1ffc0 Compare October 31, 2024 04:38

afuller-TT marked this pull request as ready for review October 31, 2024 04:41

afuller-TT requested review from tt-rkim, ttmchiou and TT-billteng as code owners October 31, 2024 04:41

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from 4a1ffc0 to aa90c2a Compare October 31, 2024 04:41

#0: Workflow to test compatibility with TTNN Project Template

cec0440

afuller-TT force-pushed the afuller/cpp-ttnn-project-test branch from aa90c2a to cec0440 Compare October 31, 2024 17:42

afuller-TT merged commit bd62962 into main Oct 31, 2024
8 checks passed

afuller-TT deleted the afuller/cpp-ttnn-project-test branch October 31, 2024 17:44

tt-rkim reviewed Oct 31, 2024

View reviewed changes

afuller-TT mentioned this pull request Oct 31, 2024

#0: Use Unity build #14535

Merged

5 tasks

o2buzzle pushed a commit that referenced this pull request Nov 4, 2024

#0: Workflow to test compatibility with TTNN Project Template (#14442)

6eb5cda

ct-clmsn pushed a commit to ct-clmsn/tt-metal that referenced this pull request Nov 12, 2024

#0: Workflow to test compatibility with TTNN Project Template (tensto…

1a65af6

…rrent#14442)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#0: Workflow to test compatibility with TTNN Project Template #14442

#0: Workflow to test compatibility with TTNN Project Template #14442

afuller-TT commented Oct 29, 2024 •

edited

Loading

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

github-actions bot left a comment

tt-rkim Oct 31, 2024

afuller-TT Oct 31, 2024

#0: Workflow to test compatibility with TTNN Project Template #14442

#0: Workflow to test compatibility with TTNN Project Template #14442

Conversation

afuller-TT commented Oct 29, 2024 • edited Loading

Ticket

Problem description

What's changed

Checklist

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

tt-rkim Oct 31, 2024

Choose a reason for hiding this comment

afuller-TT Oct 31, 2024

Choose a reason for hiding this comment

afuller-TT commented Oct 29, 2024 •

edited

Loading