PySyft icon indicating copy to clipboard operation
PySyft copied to clipboard

🔥 Hot-reload Syft nodes in your notebooks

Open itstauq opened this issue 1 year ago • 2 comments
trafficstars

Description

An alternative approach to #8971 and #8987

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

itstauq avatar Jun 27 '24 22:06 itstauq

This is incredible, beyond amazing!

snwagh avatar Jun 28 '24 17:06 snwagh

@madhavajay there are instances when we run multiple nodes in the same notebook, but are actively developing on just one node. For example, in tauquir/enclave-prototype/03-ds-execute-code.ipynb we are usually actively working on the canada-enclave node but need the other two domain nodes active as well. For such cases, it makes sense to have a separate hot_reload: bool flag in orchestra.launch instead of tying the hot reloading functionality with dev_mode=True. What are your thoughts?

itstauq avatar Jun 29 '24 18:06 itstauq

Tested this functionality by changing

  1. NodeMetadata
  2. UserService Fields
  3. Client Side Fields In jupyter lab. Screenshot 2024-07-03 at 5 13 50 PM

rasswanth-s avatar Jul 03 '24 11:07 rasswanth-s