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.

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

CSN S.A.T. Device UDP #610

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

CSN S.A.T. Device UDP #610

wants to merge 4 commits into from

Conversation

aa5sh
Copy link

@aa5sh aa5sh commented Feb 26, 2025

I've recently starting to try and operate Satellites. I am using one of these devices. It has an option to perform logging as well as can either export log entries or send a UDP blast. It has an option it says is for MacLoggerDX over port 2237. Which made me think QLog should be able to grab it. But looks like it usings 99% the format of WSJT-X. I made a condition to try and detect if the UDP from the CSN device and try to account for it not reading the adif_text part of the blast. I did also open a support request with the vendor to see if they may could make an adjustment so it would more align with the WSJT-X. From looking at it, it appears it may be missing a " before the <ADIF_VER> tag.
WSJT-X
Pasted Graphic
CSN SAT
Pasted Graphic 1

If they are able to make it work on this end I will kill this Pull request, if not I would like for it to be considered.

I built a test version so you can ignore the entitlements and action file.

Thanks

Trying to make it work to receive log entries from CSN Sat device.
m
@foldynl
Copy link
Owner

foldynl commented Feb 26, 2025

Is there a better way to detect this than to search the datagram like this bool IsCSNSat = datagram.data().contains("CSN Sat");?
I know it's unlikely, but this is dangerous in my opinion in the case that if someone gives this string to WSTJX's notes, it will detect it incorrectly. It would be better to determine where the string is expected in the received message.

@aa5sh
Copy link
Author

aa5sh commented Feb 26, 2025 via email

Moved the IF statement to only be in the mtype 12 and to further define the programid to limit any possible false positives.
removed unused variable
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.

2 participants