-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add a dynamic_reference
seedtype
#21
Comments
3 tasks
jkomoros
added a commit
that referenced
this issue
Jul 9, 2023
It generates a packed reference to a seed. It will be useful for `dynamic`. Part of #21.
jkomoros
added a commit
that referenced
this issue
Jul 9, 2023
jkomoros
added a commit
that referenced
this issue
Jul 9, 2023
… actively. Part of #21. There aren't tests yet because tests of remote seed fetches just throws.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This would allow calculating the ID to include
Type:
dynamic
.A
reference
property that expects an object, and will interpret it like a SeedReference. (Wait, won't the engine interpret a reference as a reference to execute?). Maybereference
should be a packedSeedReference. (That has the benefit that it's not interpeted by the rest of the machinery as a reference)Make sure the use case of "pass in a seed graph root to execute" works.
Update documentation for things like
seed.references()
to make it clear that it does not include dynamic seed references.dynamic should have a
allow_remote
property. Unless it's set to true, then a reference that goes to a remote packet won't load.reference
, which takesseed_id
andpacket
, and auto-merges the current location of the seed and returns a packedReference.dynamic
, which takes a reference (test to make sure it rejects remote callsdynamic
in greenallow_remote
optional boolean to dynamic.disallow_remote
that seeds can't read back, but sub-seeds below that point in environment will throw if a remote is tried to access even ifallow_remote
. And that's true for seed references that are directfetch
seed_type #45, just withdisallow_fetch
)The text was updated successfully, but these errors were encountered: