Deprecate system arg from function/module factory functions

[carolineechen]

previously, specifying system in factory function would send the function/module to the system upon construction. this behavior was updated recently to support constructing a function/module object that already lives on the cluster, such that the object's system attribute was being set but nothing was being sent. however, this is confusing for cases like rh.function(local_fn, system=remote_cluster), as no action is done and the function lives locally.

for clarity, force users to use .to(system) for sending function code to a cluster. or .from_name or rh.function(name="name") for reloading existing function/module that lives remotely.

[carolineechen]

• Deprecate system arg from function/module factory functions #625 👈
• Remove deprecated items #624
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @carolineechen and the rest of your teammates on Graphite