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.

Sign up for GitHub

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

Jump to bottom

Combine test_infra tests #1186

Closed
nsmithtt opened this issue Nov 6, 2024 · 0 comments
Closed

Combine test_infra tests #1186

nsmithtt opened this issue Nov 6, 2024 · 0 comments
Assignees
@ctodTT

Comments

@nsmithtt
Copy link
Contributor

nsmithtt commented Nov 6, 2024

We should combine the following test files:

  • python/test_infra/test_ttir_ops_ttmetal.py
  • python/test_infra/test_ttir_ops_ttnn.py

The tests by default should be backend agnostic and we just have kwargs on the decorator that defaults to targets=["ttnn", "ttmetal"], each test can then decide if it wants to target a particular backend.
ctodTT added a commit that referenced this issue Nov 12, 2024
@ctodTT
Combine test_infra Tests (#1186)
89efc3b 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 12, 2024
@ctodTT
Combine test_infra Tests (#1186)
27841f6 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 12, 2024
@ctodTT
Combine test_infra Tests (#1186)
b4ccece 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 12, 2024
@ctodTT
Combine test_infra Tests (#1186)
9e05e53 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 13, 2024
@ctodTT
Combine test_infra Tests (#1186)
c84240d 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 13, 2024
@ctodTT
Combine test_infra Tests (#1186)
900572f 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 13, 2024
@ctodTT
Combine test_infra Tests (#1186)
393ea2f 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
ctodTT added a commit that referenced this issue Nov 13, 2024
@ctodTT
Combine test_infra Tests (#1186)
404cad5 
This change combines does two main things:
	- In `python/test_infra/test_ttir_ops_tt{nn,metal}.py`, each
	  test function used to have 3 decorators that always went
	  together. These have been combined into a single decorator.

	- `python/test_infra/test_ttir_ops_tt{nn,metal}.py` have been
	  combined into a single file, with an additional parameter
	  `targets` being used to select which backend is tested for.
@ctodTT ctodTT closed this as completed Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ctodTT ctodTT

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nsmithtt @ctodTT