Skip to content

fix performance test #1884

Merged
elronbandel merged 1 commit intomainfrom
fix_performance_test
Jul 30, 2025
Merged

fix performance test #1884
elronbandel merged 1 commit intomainfrom
fix_performance_test

Conversation

@dafnapension
Copy link
Copy Markdown
Collaborator

@dafnapension dafnapension commented Jul 30, 2025

through removal from performance evaluation of _source_to_dataset, which generated a 403 to the dataset of recipe cfpb_product_2023, although the access to this dataset is free no problem:

image

@dafnapension dafnapension force-pushed the fix_performance_test branch from 04957a3 to 2f16a01 Compare July 30, 2025 06:23
Signed-off-by: dafnapension <dafnashein@yahoo.com>
@dafnapension dafnapension force-pushed the fix_performance_test branch from 2f16a01 to 12e89da Compare July 30, 2025 09:31
Copy link
Copy Markdown
Member

@elronbandel elronbandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Dafna! I also like your notebook screenshot approach!

@elronbandel elronbandel merged commit 29b7512 into main Jul 30, 2025
18 checks passed
@elronbandel elronbandel deleted the fix_performance_test branch July 30, 2025 14:01
@bnayahu
Copy link
Copy Markdown
Collaborator

bnayahu commented Jul 31, 2025

@dafnapension Shalom Dafna! Please consider reverting this change, once #1888 is merged.

@dafnapension
Copy link
Copy Markdown
Collaborator Author

dafnapension commented Jul 31, 2025 via email

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.

3 participants