8000 Create CODEOWNERS by mgkwill · Pull Request #2 · lava-nc/lava · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Create CODEOWNERS #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

Merged
merged 1 commit into from
Oct 25, 2021
Merged

Create CODEOWNERS #2

merged 1 commit into from
Oct 25, 2021

Conversation

mgkwill
Copy link
Contributor
@mgkwill mgkwill commented Oct 25, 2021

Add @lava-nc/lava-committers to CODEOWNERS ensuring
only committers can merge patches for lava-nc/lava

Add @lava-nc/lava-committers to CODEOWNERS ensuring
only committers can merge patches for lava-nc/lava
@mgkwill mgkwill requested a review from a team October 25, 2021 17:18
@awintel
Copy link
Contributor
awintel commented Oct 25, 2021

What do I need to do here?

@joyeshmishra joyeshmishra self-requested a review October 25, 2021 17:35
@mgkwill mgkwill merged commit 565f248 into lava-nc:main Oct 25, 2021
@mgkwill mgkwill deleted the add-code-owners branch October 25, 2021 17:37
@mgkwill
Copy link
Contributor Author
mgkwill commented Oct 25, 2021

Thanks @awintel @joyeshmishra

srrisbud added a commit that referenced this pull request Nov 18, 2021
* Added @tags decorator to tag ProcessModels and distinguish them

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixed @tags decorator after pull request review

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: bit-accurate with Loihi and floating point ProcessModels

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* lava -> src/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* tests -> tests/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* tests -> tests/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Manual merge of changes from message passing refactoring (after lava->src/lava reorg)

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Loihi1SimCfg handles subProcs and tags. Tutorials 2-6 work with new LIF

* Minor: builder support for float LavaPyType corrected

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: incorporated code-review suggestions

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Manually merge latest lava-nc/lava:main to the personal fork:main

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixing flake8 flagged line overflows

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixing flake8 flagged line overflows

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: changes in response to @mathisrichter's review

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

Co-authored-by: drager-intel <danielle.rager@intel.com>
mgkwill added a commit that referenced this pull request Jul 6, 2022
Update ci-build.yml use Linux only for now.
monkin77 pushed a commit to monkin77/thesis-lava that referenced this pull request Jul 12, 2024
Add @lava-nc/lava-committers to CODEOWNERS ensuring
only committers can merge patches for lava-nc/lava
monkin77 pushed a commit to monkin77/thesis-lava that referenced this pull request Jul 12, 2024
…va-nc#70)

* Added @tags decorator to tag ProcessModels and distinguish them

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixed @tags decorator after pull request review

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: bit-accurate with Loihi and floating point ProcessModels

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* lava -> src/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* tests -> tests/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* tests -> tests/lava: reorg to resolve merge conflicts

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Manual merge of changes from message passing refactoring (after lava->src/lava reorg)

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Loihi1SimCfg handles subProcs and tags. Tutorials 2-6 work with new LIF

* Minor: builder support for float LavaPyType corrected

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: incorporated code-review suggestions

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Manually merge latest lava-nc/lava:main to the personal fork:main

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Minor change to variable builder to support floating point variables created through LavaPyType

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixing flake8 flagged line overflows

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* Fixing flake8 flagged line overflows

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

* LIF: changes in response to @mathisrichter's review

Signed-off-by: Risbud, Sumedh <sumedh.risbud@intel.com>

Co-authored-by: drager-intel <danielle.rager@intel.com>
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
0