Skip to content

simplify and standardize#13

Merged
emmazhou merged 6 commits intomainfrom
emma-reorg
May 21, 2025
Merged

simplify and standardize#13
emmazhou merged 6 commits intomainfrom
emma-reorg

Conversation

@emmazhou
Copy link
Copy Markdown
Contributor

@emmazhou emmazhou commented May 20, 2025

directory structure is now like this:

.
├── CMakeLists.txt
├── CMakePresets.json
├── Dockerfile
├── LICENSE
├── README.md
├── build
├── client
│   └── listen_to_joystick.py
├── cmake
│   └── protos.cmake
├── config
│   └── simulator_32ch.json
├── dist
│   └── synapse-example-app_0.1.0_arm64.deb
├── external
├── keys
│   └── science-repo-public.asc
├── manifest.json
├── src
│   ├── fixed_weight_decoder.cpp
│   └── fixed_weight_decoder.hpp
└── vcpkg.json

to test build and deploy, you will need synapsectl changes from sciencecorp/synapse-python#114

@emmazhou emmazhou requested a review from gilbert-sci May 20, 2025 22:16
Copy link
Copy Markdown
Contributor

@gilbert-sci gilbert-sci left a comment

Choose a reason for hiding this comment

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

In the move for the decoder, you need to remove the apps/ prefix for the header you include.

Otherwise this build and deployed for me.

@emmazhou emmazhou marked this pull request as ready for review May 20, 2025 23:30
Comment thread CMakePresets.json Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

maybe include a pyproject.toml or requirements.txt for this so its easy to install all the required libraries? Up to you if that is worthwhile

Copy link
Copy Markdown

@polymerizedsage polymerizedsage left a comment

Choose a reason for hiding this comment

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

seems good

@emmazhou emmazhou merged commit 49dc750 into main May 21, 2025
@emmazhou emmazhou deleted the emma-reorg branch May 21, 2025 00:05
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.

4 participants