8000 feat: add CLI arguments for device connection by avionicharshit-byte · Pull Request #323 · Serial-Studio/Serial-Studio · GitHub
[go: up one dir, main page]
Skip to content

feat: add CLI arguments for device connection #323

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
Merged
alex-spataru merged 1 commit into from
Nov 24, 2025
Merged

feat: add CLI arguments for device connection #323

alex-spataru merged 1 commit into from
Nov 24, 2025

Conversation

@avionicharshit-byte
Copy link
Contributor
@avionicharshit-byte avionicharshit-byte commented Nov 20, 2025

Adds command-line argument support for launching Serial Studio with automatic device connection and project loading.

Feature Request #266

Changes

  • Replaced basic argument parsing with QCommandLineParser in app/src/main.cpp
  • Added --device <path> to specify serial device
  • Added --baud <rate> to set baud rate
  • Added --project <file> to load project file on startup
  • Maintained backward compatibility with existing -v/-r flags

Usage Examples

# Connect to serial device
./SerialStudio --device /dev/ttyUSB0 --baud 115200

# Load project file
./SerialStudio --project myproject.json

# Combine both
./SerialStudio --project myproject.json --device /dev/ttyUSB0 --baud 115200

Testing

- ✅ Code verified against existing APIs
- ✅ Backward compatibility maintained
- ⏳ Requires more testing with actual hardware/project files

Notes

- Device connection is established automatically before the event loop starts
- Invalid baud rates display a warning but don't prevent startup

@CLAassistant
Copy link
CLAassistant commented Nov 20, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@avionicharshit-byte avionicharshit-byte mentioned this pull request Nov 20, 2025
Open
@avionicharshit-byte
Copy link
Contributor Author
avionicharshit-byte commented Nov 22, 2025

@alex-spataru can you review this ?

@alex-spataru
Copy link
Member
alex-spataru commented Nov 24, 2025

Thank you so much for this!

I plan to expand on this code in the future to add support for device listing and selection for different IO Drivers, it definitely would help for many usage cases.

Merging...

@alex-spataru alex-spataru merged commit c4961aa into Serial-Studio:master Nov 24, 2025
2 checks passed
@recursivenomad recursivenomad mentioned this pull request Dec 15, 2025
Open
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.

3 participants

@avionicharshit-byte @CLAassistant @alex-spataru
0