Skip to content
This repository was archived by the owner on May 7, 2025. It is now read-only.

Consistently apply MIT OR Apache-2.0 license and clarify licence on test files #150

Merged
merged 2 commits into from
Mar 25, 2023

Conversation

pixelspark
Copy link
Collaborator

#149

@haixuanTao , @SludgePhD , @CameronTofer , @zimond , @aliemjay , @philpax , @maekawatoshiki please confirm you agree to dual-licensing your contributions as "MIT OR Apache-2.0" as clarified in this pull request.

@pixelspark pixelspark requested a review from haixuanTao March 19, 2023 11:38
@pixelspark pixelspark mentioned this pull request Mar 19, 2023
@pixelspark pixelspark linked an issue Mar 19, 2023 that may be closed by this pull request
@philpax
Copy link
Contributor

philpax commented Mar 19, 2023

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to choose either at their option.

@maekawatoshiki
Copy link
Contributor

maekawatoshiki commented Mar 19, 2023

I agree to dual-license my past & future contributions as MIT OR Apache-2.0.

@haixuanTao
Copy link
Collaborator

OK, for both licenses MIT or APACHE as well for both past and future.

@SludgePhD
Copy link
Contributor

I agree to dual-license my past & future contributions as MIT OR Apache-2.0.

@aliemjay
Copy link
Contributor

I agree to license my work under MIT OR Apache-2.0

1 similar comment
@zimond
Copy link
Contributor

zimond commented Mar 20, 2023

I agree to license my work under MIT OR Apache-2.0

@pixelspark
Copy link
Collaborator Author

pixelspark commented Mar 20, 2023

OK, so that leaves @CameronTofer . Can you agree to dual-license your contribution (#139, see your commits )?

@pixelspark pixelspark assigned pixelspark and unassigned haixuanTao Mar 20, 2023
@pixelspark pixelspark removed the request for review from haixuanTao March 20, 2023 23:01
@pixelspark pixelspark removed their assignment Mar 21, 2023
This reverts commit 9fc367f, reversing
changes made to eaf889e.

# Conflicts:
#	wonnx/src/compiler.rs
#	wonnx/templates/matrix/lrn.wgsl
#	wonnx/tests/localresponsenormalization.rs
@pixelspark
Copy link
Collaborator Author

pixelspark commented Mar 25, 2023

We are now licensed "MIT OR Apache-2.0" ! Thanks all for the replies.

@CameronTofer unfortunately I had to remove your contribution for now. Creating certainty about the licensing situation is in the best interest of the project. If you agree to relicense your contributions we can simply merge it back (I prepared a PR: #152).

@antimora curious to see what you will be using in burn-rs!

@antimora
Copy link

@pixelspark Awesome! Thank you for the clarification.

We would like to use your code in the Burn project as a burn-wgpu backend. Currently, Burn has two backends (torch and ndarray). The burn-wgpu backend will allow accessing GPU on my many platforms without the need to install torch. This is great for training and inference. Also, burn-wgpu will allow acceleration on the web. Just recently we published an example running MNIST inference on the web using wasm. You can try the live demo. So if we have a burn-wgpu backend, it will be easy just to swap the burn-ndarray backend with burn-wgpu, to have GPU acceleration on the web.

(I am tagging @nathanielsimard here so he's aware of this ticket. He's the creator and developer of Burn. He was also excited about the possibility to run train/inference on AMD GPUs).

@pixelspark pixelspark deleted the chore/dual-license branch March 25, 2023 20:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

license clarification
8 participants