Skip to content

Conversation

@whitneywhtsang
Copy link
Contributor

@whitneywhtsang whitneywhtsang commented Nov 29, 2024

The performance of default path without env vars is similar to the performance of default path with env vars.
This PR removes running default path with env vars in CI.

@whitneywhtsang whitneywhtsang changed the title [CI][GEMM] Remove default path with env vars configuration [CI][GEMM][FA] Remove default path with env vars configuration Nov 29, 2024
@whitneywhtsang
Copy link
Contributor Author

@Egor-Krivov Can we update Grafana's references of ci-dflt with ci after this PR?

@whitneywhtsang whitneywhtsang linked an issue Nov 29, 2024 that may be closed by this pull request
@Egor-Krivov
Copy link
Contributor

@Egor-Krivov Can we update Grafana's references of ci-dflt with ci after this PR?

Let's discuss target state first

@whitneywhtsang whitneywhtsang merged commit f3ad673 into main Dec 2, 2024
6 checks passed
@whitneywhtsang whitneywhtsang deleted the whitneywhtsang/gemm_ci branch December 2, 2024 14:09
Egor-Krivov added a commit that referenced this pull request Dec 2, 2024
Get rid of the workaround from
#2882

Dashboards are already updated
#2882 (comment)
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.

Cleanup unnecessary env variables

6 participants