Skip to content
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

feat(metagen): mdk_rs finalization #673

Merged
merged 9 commits into from
Apr 30, 2024

Conversation

Yohe-Am
Copy link
Contributor

@Yohe-Am Yohe-Am commented Apr 12, 2024

Motivation and context

The code generator was already in place but we the typegate had no support for the wit_wire interface used by the rust mdk. This PR adds that.

The two week delay is mainly due to the base work required in #669 and related PRs.

Migration notes

No breaking changes on user.

Checklist

  • The change come with new or modified tests
  • Hard-to-understand functions have explanatory comments
  • End-user documentation is updated to reflect the change

@Yohe-Am Yohe-Am requested a review from michael-0acf4 April 12, 2024 07:35
Copy link
Contributor

@michael-0acf4 michael-0acf4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@Yohe-Am Yohe-Am force-pushed the feat/mdk-rs-second-leg branch from 8cdef28 to 731fa3e Compare April 28, 2024 18:42
@Yohe-Am Yohe-Am changed the base branch from main to upload/deno-runtime April 28, 2024 18:46
@Yohe-Am Yohe-Am changed the base branch from upload/deno-runtime to refactor/MET-404/wasmtime-pyrt April 28, 2024 18:47
.github/workflows/tests.yml Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
ghjk.ts Show resolved Hide resolved
typegate/src/runtimes/mod.ts Outdated Show resolved Hide resolved
typegate/src/runtimes/wasm.ts Outdated Show resolved Hide resolved
@Yohe-Am Yohe-Am marked this pull request as ready for review April 28, 2024 19:11
@Yohe-Am Yohe-Am mentioned this pull request Apr 28, 2024
3 tasks
Copy link
Member

@zifeo zifeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 (needs a second-longer look but already some feedbacks)

libs/metagen/src/mdk_rust/static/lib.rs Show resolved Hide resolved
libs/metagen/tests/mat_rust/mdk.rs Outdated Show resolved Hide resolved
typegate/tests/runtimes/python/python_test.ts Outdated Show resolved Hide resolved
typegate/tests/runtimes/wasm_wire/rust.wasm Outdated Show resolved Hide resolved
ghjk.ts Show resolved Hide resolved
@Yohe-Am Yohe-Am force-pushed the refactor/MET-404/wasmtime-pyrt branch from 30ff471 to 93e1a7d Compare April 29, 2024 19:33
@Yohe-Am Yohe-Am force-pushed the feat/mdk-rs-second-leg branch from 731fa3e to 6019011 Compare April 30, 2024 07:47
Yohe-Am added 3 commits April 30, 2024 09:27
Implements python runtime support for `import_function` materializers.
This is an stacked PR on #673 which itself is appended on #687 for
tackling MET-404.

#### Migration notes

This PR introduced a breaking change in the importer logic for python
modules brought in through `pyrt.import_` and it's deps. After this PR,
any path imports must be prefixed with `.`, `..`, `...` and so to point
to relative path locations. This behavior is at odds with standard
python which looks for relative modules and packages even if there are
no dot prefixes.

- [x] The change come with new or modified tests
- [x] Hard-to-understand functions have explanatory comments
<!-- 5. Readiness checklist
- [ ] End-user documentation is updated to reflect the change
-->
@Yohe-Am Yohe-Am merged commit 3d989a6 into refactor/MET-404/wasmtime-pyrt Apr 30, 2024
6 of 11 checks passed
@Yohe-Am Yohe-Am deleted the feat/mdk-rs-second-leg branch April 30, 2024 16:21
Yohe-Am pushed a commit that referenced this pull request May 7, 2024
fix: cherry-pick

feat(wasm): wit component support

test(wasm): error propagation

refactor(wasm): all remaining wasmedge -> wasm

fix(wasm): bad conversion

feat(wasm): nested object output

test(wasm): tuple deserialize

test(wasm): update binding test value

feat(wasm): nested wit input support

feat(wasm): handle enum input, fix object bug

feat(wasm): reject on extra fields

fix(wasm): typos and minor cleanups

fix(tests.yml): disable cache deno dir for now

fix: version lock

fix: lockfile

refactor: rename all wasmedge ref. to wasm

refactor(gate): wasi 0.2 pyrt

refactor: rename to `pyrt_wit_wire`

fix: pre-commit issue

wip: try ci fix

feat: add pyrt bin inline

refactor: `python_wasi` -> `python`

wip: import module support

fix: rebase bugs

feat(metagen): `mdk_rs` finalization (#673)

- Finalizes the host side of the `wit_wire` implementation started in
- Fixes a few issues with the `mdk_rust` generator.

The code generator was already in place but we the typegate had no
support for the `wit_wire` interface used by the rust mdk. This PR adds
that.

The two week delay is mainly due to the base work required in #669 and
related PRs.

_No breaking changes on user._

- [x] The change come with new or modified tests
- [x] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change

feat: Enable batch prisma queries in the typegate runtime (#682)

Enable batch prisma queries (and transaction) in the typegate runtime

Console

[MET-381](https://linear.app/metatypedev/issue/MET-381/console-collections)

<!-- Explain HOW users should update their code when required -->

- [x] The change come with new or modified tests
- [ ] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change

---------

Co-authored-by: Teo Stocco <[email protected]>
Co-authored-by: Yohe-Am <[email protected]>

fix: update poetry lockfile

fix: CI issues

fix: minor bugs

fix: ci breakage

wip: try fix
Yohe-Am pushed a commit that referenced this pull request May 23, 2024
fix: cherry-pick

feat(wasm): wit component support

test(wasm): error propagation

refactor(wasm): all remaining wasmedge -> wasm

fix(wasm): bad conversion

feat(wasm): nested object output

test(wasm): tuple deserialize

test(wasm): update binding test value

feat(wasm): nested wit input support

feat(wasm): handle enum input, fix object bug

feat(wasm): reject on extra fields

fix(wasm): typos and minor cleanups

fix(tests.yml): disable cache deno dir for now

fix: version lock

fix: lockfile

refactor: rename all wasmedge ref. to wasm

refactor(gate): wasi 0.2 pyrt

refactor: rename to `pyrt_wit_wire`

fix: pre-commit issue

wip: try ci fix

feat: add pyrt bin inline

refactor: `python_wasi` -> `python`

wip: import module support

fix: rebase bugs

feat(metagen): `mdk_rs` finalization (#673)

- Finalizes the host side of the `wit_wire` implementation started in
- Fixes a few issues with the `mdk_rust` generator.

The code generator was already in place but we the typegate had no
support for the `wit_wire` interface used by the rust mdk. This PR adds
that.

The two week delay is mainly due to the base work required in #669 and
related PRs.

_No breaking changes on user._

- [x] The change come with new or modified tests
- [x] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change

feat: Enable batch prisma queries in the typegate runtime (#682)

Enable batch prisma queries (and transaction) in the typegate runtime

Console

[MET-381](https://linear.app/metatypedev/issue/MET-381/console-collections)

<!-- Explain HOW users should update their code when required -->

- [x] The change come with new or modified tests
- [ ] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change

---------

Co-authored-by: Teo Stocco <[email protected]>
Co-authored-by: Yohe-Am <[email protected]>

fix: update poetry lockfile

fix: CI issues

fix: minor bugs

fix: ci breakage

wip: try fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants