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.

Sign up for GitHub

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

Jump to bottom

Let models have multiple inputs and outputs #218

Merged
merged 24 commits into from
Jul 14, 2021
Merged

Let models have multiple inputs and outputs #218

merged 24 commits into from
Jul 14, 2021

Conversation

Michael-F-Bryan
Copy link
Contributor

Fixes #164.

@Michael-F-Bryan Michael-F-Bryan force-pushed the feature/model-multiple-inputs-and-outputs branch 3 times, most recently from 9ba821a to d6f8d1a Compare July 13, 2021 14:19
@Michael-F-Bryan Michael-F-Bryan mentioned this pull request Jul 14, 2021
Open
Michael-F-Bryan added 24 commits July 15, 2021 05:40
Added filtering to the integration test suite
d4a2102
Command failures will now record stdout and stderr
970ed29
Added a basic failing test
eff1fe2
Created a TensorRepr type
afbf97c
Deleted the previous TensorRepr code
dd4e1e6
Added a Shape type
da70db6
Created a model type which can accept multiple inputs and multiple ou…
a371189 
…tputs
Simplified our TensorList a bit
cf4daa8
Added mimetypes to the loading host function
ff03d5d
Assume all tests should be run if no filter is provided
5833c1c
Wired multiple inputs/outputs up to codegen
44bf975
The generated code now compiles!
010207f
Added a style transfer Rune
5ff3e4d
Tweaked our log levels to not log for external deps by default
bc96452
Introduced a new image type
a9b90dc
Wiring the new image type up to wasmer directly
3aaad86
Wired up most of the model loading and inference
7b1858c
Wired up all host functions
6e51b5f
Removed the extra level of indirection from our "Function" type
b9a9322
Gave the Serial output a resizeable buffer
fda5c0b
Got the FFI bindings compiling again
b29a6c9
Gave the Tensor type a From impl for larger arrays
c1639c5
Fixing up the remaining tests
3ac81c9
Integration tests now pass
f364c54
@Michael-F-Bryan Michael-F-Bryan force-pushed the feature/model-multiple-inputs-and-outputs branch from 4b89eb0 to f364c54 Compare July 14, 2021 21:40
@Michael-F-Bryan Michael-F-Bryan merged commit 0ccd616 into master Jul 14, 2021
@Michael-F-Bryan Michael-F-Bryan deleted the feature/model-multiple-inputs-and-outputs branch July 14, 2021 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Change host functions to allow models to have multiple inputs/outputs
1 participant
@Michael-F-Bryan