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

Clarying a lot the use of deployment for Breast Cancer [WIP] #893

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Clarying a lot the use of deployment for Breast Cancer [WIP] #893

wants to merge 1 commit into from

Conversation

bcm-at-zama
Copy link
Collaborator

@bcm-at-zama bcm-at-zama commented Sep 27, 2024 •
edited
Loading

I have clarified a bit and checked it works fine on my mac. Maybe to be merged with the other PR, ie #876

@bcm-at-zama bcm-at-zama requested a review from a team as a code owner September 27, 2024 14:42
@cla-bot cla-bot bot added the cla-signed label Sep 27, 2024
andrei-stoian-zama
andrei-stoian-zama requested changes Sep 27, 2024
View reviewed changes
Copy link
Collaborator

@andrei-stoian-zama andrei-stoian-zama left a comment

Choose a reason for hiding this comment

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

Please keep the port 5000. Or maybe if you find a port that always works on MAC that's good too

@bcm-at-zama bcm-at-zama force-pushed the checking_breast_cancer_deployment_on_my_mac branch 6 times, most recently from 760584b to b71b8f5 Compare September 27, 2024 16:13
@bcm-at-zama bcm-at-zama changed the title fix: fixing the call to train.py directly in python Clarying a lot the use of deployment for Breast Cancer Sep 27, 2024
@bcm-at-zama bcm-at-zama force-pushed the checking_breast_cancer_deployment_on_my_mac branch 6 times, most recently from ea07869 to a7fcee8 Compare September 27, 2024 18:01
@bcm-at-zama
fix: fixing the call to train.py directly in python
238070e 
else we have issues with the versions
@bcm-at-zama bcm-at-zama force-pushed the checking_breast_cancer_deployment_on_my_mac branch from a7fcee8 to 238070e Compare September 27, 2024 18:19
@bcm-at-zama bcm-at-zama changed the title Clarying a lot the use of deployment for Breast Cancer Clarying a lot the use of deployment for Breast Cancer [WIP] Sep 28, 2024
bcm-at-zama
bcm-at-zama commented Sep 28, 2024
View reviewed changes
use_case_examples/deployment/breast_cancer/client_via_tfhe-rs.py
@@ -0,0 +1,208 @@
"""Client script.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ill remove this file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrei-stoian-zama andrei-stoian-zama andrei-stoian-zama requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bcm-at-zama @andrei-stoian-zama