-
Notifications
You must be signed in to change notification settings - Fork 109
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
Re-enable use of memory.grow
on drun
#694
Conversation
requires https://github.com/dfinity-lab/dfinity/pull/1270 Also use a `drun` config file to set absurd gas limits.
now that https://github.com/dfinity-lab/dfinity/pull/1270 is merged.
The test case takes 1m32s on my laptop. (partly because we also run it interpreted three times). Not sure if that’s worth it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
default.nix
Outdated
@@ -33,7 +33,7 @@ let dev = import (builtins.fetchGit { | |||
let dfinity-repo = import (builtins.fetchGit { | |||
url = "ssh://git@github.com/dfinity-lab/dfinity"; | |||
ref = "master"; | |||
rev = "a50367859416ef7c12ca103b7fc03f5a7654f6ad"; | |||
rev = "1b7e7c185fdd20a61b13e4c76d4dde29dce9862e"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need a bump (probably).
Is that because of the high limit (allocation costs)? Why not reduce it to a reasonable size then? |
Oh, I didn’t get this in? Thanks for pinging me. |
## Changelog for motoko-base: Branch: next-moc Commits: [dfinity/motoko-base@8fd23fa7...a3dd04eb](dfinity/motoko-base@8fd23fa...a3dd04e) * [`cee15564`](dfinity/motoko-base@cee1556) feat: add `isReplicated` to `ExperimentalInternetComputer` ([dfinity/motoko-base#694](https://togithub.com/dfinity/motoko-base/issues/694)) * [`c9a08167`](dfinity/motoko-base@c9a0816) feat: add `Error.isRetryPossible`, a predicate for non-fatal messaging errors ([dfinity/motoko-base#692](https://togithub.com/dfinity/motoko-base/issues/692)) * [`6246b654`](dfinity/motoko-base@6246b65) feat: make `replyDeadline` an optional return type ([dfinity/motoko-base#693](https://togithub.com/dfinity/motoko-base/issues/693)) * [`1a381b99`](dfinity/motoko-base@1a381b9) Motoko 0.14.3 * [`756855bc`](dfinity/motoko-base@756855b) chore: amend the 0.14.3 changelog ([dfinity/motoko-base#697](https://togithub.com/dfinity/motoko-base/issues/697))
## Changelog for motoko-base: Branch: next-moc Commits: [dfinity/motoko-base@8fd23fa7...a3dd04eb](dfinity/motoko-base@8fd23fa...a3dd04e) * [`cee15564`](dfinity/motoko-base@cee1556) feat: add `isReplicated` to `ExperimentalInternetComputer` ([dfinity/motoko-base#694](https://togithub.com/dfinity/motoko-base/issues/694)) * [`c9a08167`](dfinity/motoko-base@c9a0816) feat: add `Error.isRetryPossible`, a predicate for non-fatal messaging errors ([dfinity/motoko-base#692](https://togithub.com/dfinity/motoko-base/issues/692)) * [`6246b654`](dfinity/motoko-base@6246b65) feat: make `replyDeadline` an optional return type ([dfinity/motoko-base#693](https://togithub.com/dfinity/motoko-base/issues/693)) * [`1a381b99`](dfinity/motoko-base@1a381b9) Motoko 0.14.3 * [`756855bc`](dfinity/motoko-base@756855b) chore: amend the 0.14.3 changelog ([dfinity/motoko-base#697](https://togithub.com/dfinity/motoko-base/issues/697))
requires https://github.com/dfinity-lab/dfinity/pull/1270
Also use a
drun
config file to set absurd gas limits.