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

Add NxSDKRuntimeService #182

Merged
merged 22 commits into from
Feb 25, 2022
Merged

Add NxSDKRuntimeService #182

merged 22 commits into from
Feb 25, 2022

Conversation

mgkwill
Copy link
Contributor

@mgkwill mgkwill commented Feb 15, 2022

Issue Number: #181

Objective of pull request: Create NxSDKRuntimeService

Pull request checklist

Your PR fulfills the following requirements:

Pull request type

Please check your PR type:

  • Feature

What is the current behavior?

  • NxSDKRuntimeService does not exist and Lava cannot interact with Loihi.

What is the new behavior?

  • NxSDKRuntimeService is able to interact with Loihi through NxCore.

Does this introduce a breaking change?

  • No

mgkwill and others added 4 commits February 15, 2022 13:07
@mgkwill
Address review comments
19bb4b4 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@srrisbud @PhilippPlank @mgkwill
Improve mnist tutorial (lava-nc#147)
1afbf35 
* Minor: Removed a divide by zero warning from the fixed point LIF ProcessModel

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

* Improved to MNIST end-to-end tutorial
    - uses fixed point bit-accurate ProcessModels for LIF and Dense
    - resets internal neural state of all LIF neurons
    - these changes are needed to make the pre-trained networks parameters work, because the network was trained with these assumptions

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

* Post code review @awintel and @phstratmann

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

* Post code review @awintel and @phstratmann

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

* Post re-review by @phstratmann

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

Co-authored-by: PhilippPlank <32519998+PhilippPlank@users.noreply.github.com>
@mgkwill
Check process lineage before join (lava-nc#177)
63cb4ca 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill
Add NxSDKRuntimeService
5aa3985 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill mgkwill added area: magma/runtime Issues with something in lava/magma/runtime area: magma/compiler Issues with something in lava/magma/compiler labels Feb 15, 2022
@mgkwill mgkwill self-assigned this Feb 15, 2022
@mgkwill mgkwill linked an issue Feb 15, 2022 that may be closed by this pull request
Create NxSDKRuntimeService #181
Closed
2 tasks
@mgkwill
Fix unit test, linting
e6f3785 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill
Remove comments
4ea87a8 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill
Copy link
Contributor Author

mgkwill commented Feb 15, 2022 •
edited

Add docstrings everywhere.

@mgkwill
Handle nxsdk import exception
b7c4271 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill
Copy link
Contributor Author

mgkwill commented Feb 15, 2022

NxSDKRuntimeService does not meet class naming standards.

Change to NxSdkRuntimeService.

Same everywhere for NxSDK.

@mgkwill
Fix indentation issue
b29e8eb 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
@mgkwill
Merge branch 'main' into nxsdkRTS_base
211be04
@mgkwill
Uncomment board.run in nc proc model
0c793ef 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
joyeshmishra
joyeshmishra previously requested changes Feb 16, 2022
View reviewed changes
src/lava/magma/core/model/nc/model.py Outdated Show resolved Hide resolved
src/lava/magma/core/model/nc/model.py Outdated Show resolved Hide resolved
tests/lava/magma/runtime/test_runtime_service.py Outdated Show resolved Hide resolved
awintel
awintel reviewed Feb 16, 2022
View reviewed changes
Copy link
Contributor

@awintel awintel left a comment

Choose a reason for hiding this comment

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

Before approving or requesting changes I need some fundamental architectural questions clarified.

src/lava/magma/runtime/runtime_services/interfaces.py Show resolved Hide resolved
src/lava/magma/runtime/runtime_services/runtime_service.py Outdated Show resolved Hide resolved
src/lava/magma/runtime/runtime_services/runtime_service.py Outdated Show resolved Hide resolved
src/lava/magma/core/model/nc/ports.py Outdated Show resolved Hide resolved
src/lava/magma/core/model/nc/model.py Show resolved Hide resolved
mgkwill and others added 3 commits February 17, 2022 10:28
@mgkwill
Address review, rework NxSdkRuntime Service
5fb1b44 
Signed-off-by: Marcus G K Williams <168222+mgkwill@users.noreply.github.com>
Fix unit tests, merge with main
5238466 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Merge branch 'main' into nxsdkRTS_base
7b93111
@mgkwill mgkwill dismissed joyeshmishra’s stale review February 24, 2022 00:20

Old review. Please look at update.

Remove nc/ports.py again
ef3bfe9 
Remove commented code in compiler.py

Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
mgkwill and others added 7 commits February 24, 2022 11:16
@mgkwill
Merge branch 'main' into nxsdkRTS_base
cd0cb0e
Update comments logging
1274d3e 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Update test Utils method name and document
73e8367 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Update test name and docs for nxsdkruntimeservice
382a827 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Update docstrings for RuntimeService
1991f38 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Update logging
027910d 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
Remove unneeded logging import
3b232d8 
Signed-off-by: Marcus G K Williams <Marcus G K Williams 168222+mgkwill@users.noreply.github.com>
@joyeshmishra joyeshmishra merged commit 64a3e4c into lava-nc:main Feb 25, 2022
@mgkwill mgkwill deleted the nxsdkRTS_base branch February 25, 2022 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@awintel awintel awintel left review comments

@joyeshmishra joyeshmishra joyeshmishra approved these changes

@ysingh7 ysingh7 ysingh7 approved these changes

@srrisbud srrisbud Awaiting requested review from srrisbud

@PhilippPlank PhilippPlank Awaiting requested review from PhilippPlank

Assignees

@mgkwill mgkwill

Labels
area: magma/compiler Issues with something in lava/magma/compiler area: magma/runtime Issues with something in lava/magma/runtime
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create NxSDKRuntimeService
5 participants
@mgkwill @joyeshmishra @ysingh7 @awintel @srrisbud