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

Streaming/cufe #211

Merged
merged 9 commits into from
Nov 6, 2023
Merged

Conversation

ZiaddAhmedd
Copy link
Contributor

Submitting the streaming track for the contest
Team name: cufe

@harsha-simhadri
Copy link
Owner

@ZiaddAhmedd Could you please add a CI test entry for your algo

@ZiaddAhmedd
Copy link
Contributor Author

I have added a CI test workflow cufe_CI.yml and removed the other workflows

@ZiaddAhmedd
Copy link
Contributor Author

Is there still a problem with our files?

@AbdelrahmanMohamed129
Copy link
Contributor

is there any problem considering the merge of this branch?

@harsha-simhadri
Copy link
Owner

@AbdelrahmanMohamed129 I see the following results. Are they as you expected

cufe,"diskann(('R32_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,,0.0,1.7107985019683838,4885612.0,0.0,0.0,streaming,0.7688699999999955
cufe,"diskann(('R32_L70', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,,0.0,1.7178819179534912,4885448.0,0.0,0.0,streaming,0.7937900000000012
cufe,"diskann(('R50_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,,0.0,1.9073610305786133,4885652.0,0.0,0.0,streaming,0.8189199999999953

@AbdelrahmanMohamed129
Copy link
Contributor

AbdelrahmanMohamed129 commented Nov 5, 2023

yes these are the expected results thanks

@harsha-simhadri
Copy link
Owner

@AbdelrahmanMohamed129 thanks. Could you please remove the CI testfile specifically for cufe (or move it to your algo folder for reference). once that is done, I will merge.

@AbdelrahmanMohamed129
Copy link
Contributor

i just deleted it

@harsha-simhadri harsha-simhadri merged commit f799bc8 into harsha-simhadri:main Nov 6, 2023
24 of 28 checks passed
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