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

700ms delay needed for ptt on transition #1412

Closed
mdblack98 opened this issue Oct 31, 2023 · 11 comments
Closed

700ms delay needed for ptt on transition #1412

mdblack98 opened this issue Oct 31, 2023 · 11 comments
Labels
critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. enhancement fixed Issue has been fixed needs test Patches have been submitted but need testing to close issue
Milestone

Comments

@mdblack98
Copy link
Contributor

My 10GHz set needs 700 ms to go from receive to transmit due to the waveguide relay. So I think the audio must be delayed at least 700 ms with respect to the PTT-signal.
I could not set the TX delay to a value higher than 0.5 s.
And I am worried that the delay is too long, so that the transmission is too late

@mdblack98 mdblack98 added enhancement critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. labels Oct 31, 2023
@mdblack98 mdblack98 added this to the 4.6 milestone Oct 31, 2023
mdblack98 added a commit that referenced this issue Oct 31, 2023
…nal relay operations

Detect Doppler frequency changes for Kenwood rigs and avoid querying frequency...just set it...to provide stable timing
#1412
@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 1, 2023

This commit causes me to lose PTT control from wsjt-x (on my TS-890S). Verified with git bisect.

mdblack98 added a commit that referenced this issue Nov 1, 2023
@mdblack98 mdblack98 added the needs test Patches have been submitted but need testing to close issue label Nov 1, 2023
@mdblack98
Copy link
Contributor Author

mdblack98 commented Nov 1, 2023 via email

@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 9, 2023

Still broken with current git and wsjtx 2.6.1
WSJT-X_RigControl.log

Still looks like this is caused by the additional item added to hamlib_port_t by commit 6cb17e4 "Add post_ptt_delay for situations..."

@mdblack98
Copy link
Contributor Author

mdblack98 commented Nov 9, 2023 via email

@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 9, 2023

That was with the latest, as of 2023-11-09 10:20EST

@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 9, 2023

Pulled, made, installed, ran again - still broken.

@mdblack98
Copy link
Contributor Author

mdblack98 commented Nov 9, 2023 via email

@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 9, 2023

Sorry gotta leave for appointment. Which log was that timestamp from? Today's log attached 5 comments above.

@mdblack98
Copy link
Contributor Author

mdblack98 commented Nov 9, 2023 via email

@GeoBaltz
Copy link
Contributor

GeoBaltz commented Nov 9, 2023

OK, back again.
WSJT-X_RigControl.log

Problem still exists.

Aside: bogus version info due to stale copy of hamlibdatetime.h in Hamlib/src/, which was then included instead of newly created version in Hamlib/build/src/. Dunno how it got there; this version from latest git pull;make clean;make;

@GeoBaltz
Copy link
Contributor

I see my emails still don't show up here.

Anyway, still broken with latest git master - no PTT control with wsjtx(2.6.1)
Latest debug log:
WSJT-X_RigControl.log

ISTM the problem is with the item added to struct hamlib_port, in spite of the comment/warning to not touch it until 5.0. Binary or shared library incompatibility??

GeoBaltz added a commit to GeoBaltz/Hamlib that referenced this issue Dec 4, 2023
No need for post_ptt_delay to be in hamlib_port - move it to misc
  rig_state data.
Update references to post_ptt_delay
Fix a couple of typos
Add code to actually set post_ptt_delay

Fixes issue Hamlib#1412
@mdblack98 mdblack98 added the fixed Issue has been fixed label Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. enhancement fixed Issue has been fixed needs test Patches have been submitted but need testing to close issue
Projects
None yet
Development

No branches or pull requests

2 participants