Change pointers to store absolute path and work with .to.

[rohinb2]

Only change the module pointers to the remote_import_path when actually running .to.

In that case, we generate the remote_import_path relative to whatever env it is being .to into.

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: runhouse/resources/module.py

Function	Unhandled Issue
to	[**ValueError: Need an initialized local server in order to put a module onto rh.here. Please run runhouse re...**](https://runhouse.sentry.io/issues/5540885505/?referrer=github-open-pr-bot) ... <br> Event Count:` 1

---

_{Did you find this useful? React with a 👍 or 👎}

[rohinb2]

• Change pointers to store absolute path and work with .to. #989 : 2 dependent PRs (#991 , #996 ) 👈
• Always insert at the beginning of sys.path when modifying it. #990
• Test sending a Module directly to a new env. #970 : 1 other dependent PR (#972 )
• Remove unneeded tmp_path arg. #971
• Convert to a Package when detecting if a req contains the module. #969
• Refactor & simplify logic to detect where a module is within an env. #968
• Add test for installing correct Package version. #967
• Always convert string req to Package object. #966
• Move run_with_logs into utils. #965
• Use local version of package if exists. #964
• Refactor Package._install to have clearer logic. #963
• Reorganize Package.from_string. #962
• main

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

Join @rohinb2 and the rest of your teammates on Graphite

[rohinb2]

Merge activity

• Jul 12, 4:13 PM EDT: @rohinb2 started a stack merge that includes this pull request via Graphite.
• Jul 12, 4:17 PM EDT: Graphite rebased this pull request as part of a merge.
• Jul 12, 4:18 PM EDT: @rohinb2 merged this pull request with Graphite.