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

#12658: Update sweeps export to sqlite script #12951

Merged
merged 1 commit into from
Sep 20, 2024
Merged

Conversation

bbradelTT
Copy link
Contributor

Ticket

Link to Github Issue #12658

Problem description

  • export script was out of date and didn't work

What's changed

  • updated script to handle current sweeps ElasticSearch configuration

Checklist

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

Testing

Tested with

  • various parameters for the filter string related to matmul
  • verbose on and off and
  • no parameters, which skipped over suite processing if the index (sweep_name) was not found

Sqlite tables were generated properly

@bbradelTT bbradelTT merged commit e23e894 into main Sep 20, 2024
6 checks passed
@bbradelTT bbradelTT deleted the bbradel-12658_sql branch September 20, 2024 19:38
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.

2 participants