n4af / TR4W

TRLOG 4 Windows free amateur radio logging application
GNU General Public License v3.0
19 stars 6 forks source link

Add Icom 7610 as a radio (CI-V Address 98h) #282

Closed ny4i closed 2 years ago

ny4i commented 7 years ago

The Japanese manual stated the address to be 98h. Verify the basic operation to be just as the 7610 and then add the new radio type with address 98h.

ny4i commented 6 years ago

I compared the IC7610 Japanese Supplementary manual to the English 7300 manual. The functions that TR4W uses are identical so rather than make the 7610 just like the 7600, change it to be in the same category as the 7300 code in the radio definition arrays and all should be ok. I will dig out the dev laptop (have to do that for a UDP fix for Marv) and get that coded.

n4af commented 6 years ago

RR. Will do. Tks, Howie

On Fri, Nov 24, 2017 at 2:52 PM, Tom Schaefer notifications@github.com wrote:

I compared the IC7610 Japanese Supplementary manual to the English 7300 manual. The functions that TR4W uses are identical so rather than make the 7610 just like the 7600, change it to be in the same category as the 7300 code in the radio definition arrays and all should be ok. I will dig out the dev laptop (have to do that for a UDP fix for Marv) and get that coded.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-346889131, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqxUEO4vlnpm6Vvg1ioit93h84GRDh0ks5s5x5kgaJpZM4QGVYe .

n4af commented 6 years ago

closed - build 4.65.7 adds type ic7610

ny4i commented 6 years ago

Howie, did you add it to the arrays of radio’s that support CWByCAT and CWSpeed?

Thanks

Tom

Principal Solutions Architect Better Software Solutions, Inc. 727-437-2771

On Nov 27, 2017, at 9:34 AM, Howie Hoyt notifications@github.com wrote:

closed - build 4.65.7 adds type ic7610

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-347199155, or mute the thread https://github.com/notifications/unsubscribe-auth/AL07StlKwsuRW1kEbJDxPdMSFGrU69aSks5s6shHgaJpZM4QGVYe .

ny4i commented 6 years ago

I guess I should change that to add a CWByCAT option to the radio properties array. I’m not sure why I didn’t do it that way.

Tom

Principal Solutions Architect Better Software Solutions, Inc. 727-437-2771

On Nov 27, 2017, at 9:34 AM, Howie Hoyt notifications@github.com wrote:

closed - build 4.65.7 adds type ic7610

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-347199155, or mute the thread https://github.com/notifications/unsubscribe-auth/AL07StlKwsuRW1kEbJDxPdMSFGrU69aSks5s6shHgaJpZM4QGVYe .

n4af commented 6 years ago

Ok. The changes for CWBC and speed are in my master and at https://github.com/n4af/TR4W/commit/0bd44e635a0abb220e158130758b2e10c60fddca

They will not actually make code until the next mod 4.65.8 or 4.66

73, Howie

ny4i commented 6 years ago

Howie, It looks like the IC7610 entry was simply a clone of the line for the IC7600. The address looks off as it is the same address as the IC7600. The $7A below should be $98 for the 7610.

({Name: 'IC7610'; } BR:BR9600; P: 1; C: 0; S: 0; T:0; RA: $7A; SW: 1; SFOC: $00; SMOC: $00; SPOC: $00; RX: $00; TX: $00; MB: 0; CW: $00; LSB: $00; USB: $00; FM: $00; AM: $00; DIGL: $00; DIGU: $00; rt: rtIcom), // 4.65.7 issue #282

I will check the code to see what is happening elsewhere.

n4af commented 6 years ago

Hi Tom

I think this http://tr4w.net/4.68/tr4w_setup_4_68.12.exereflects the change u suggested.

to: ({Name: 'IC7610'; } BR:BR9600; P: 1; C: 0; S: 0; T:0; RA: $98; SW: 1; SFOC: $00; SMOC: $00; SPOC: $00; RX: $00; TX: $00; MB: 0; CW: $00; LSB: $00; USB: $00; FM: $00; AM: $00; DIGL: $00; DIGU: $00; rt: rtIcom), // 4.65.7 issue #282 #282 // 4.68.12

I will not be available until Monday, as in ARRLDX-CW prep mode.

73, Howie

On Fri, Feb 16, 2018 at 9:45 AM Tom Schaefer notifications@github.com wrote:

Reopened #282 https://github.com/n4af/TR4W/issues/282.

— You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#event-1478004521, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqxUBe2P6s4mLZn3S5P9a8xrzBsMU00ks5tVZSggaJpZM4QGVYe .

ny4i commented 6 years ago

Thanks. I will test that against my 7300 as it should be the same. I think Terry might have some other issue.

GL in the contest. I have no doubt some will work me and think they worked you and vice versa (If you are using NY4A) :)

Tom Principal Solutions Architect, ITIL®v3 Better Software Solutions, Inc. 727-437-BSS1 (727-437-2771) @BSSI_Consulting

On Feb 16, 2018, at 10:39 AM, Howie Hoyt notifications@github.com wrote:

Hi Tom

I think this http://tr4w.net/4.68/tr4w_setup_4_68.12.exereflects the change u suggested.

to: ({Name: 'IC7610'; } BR:BR9600; P: 1; C: 0; S: 0; T:0; RA: $98; SW: 1; SFOC: $00; SMOC: $00; SPOC: $00; RX: $00; TX: $00; MB: 0; CW: $00; LSB: $00; USB: $00; FM: $00; AM: $00; DIGL: $00; DIGU: $00; rt: rtIcom), // 4.65.7 issue #282 #282 // 4.68.12

I will not be available until Monday, as in ARRLDX-CW prep mode.

73, Howie

On Fri, Feb 16, 2018 at 9:45 AM Tom Schaefer notifications@github.com wrote:

Reopened #282 https://github.com/n4af/TR4W/issues/282.

— You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#event-1478004521, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqxUBe2P6s4mLZn3S5P9a8xrzBsMU00ks5tVZSggaJpZM4QGVYe .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-366270147, or mute the thread https://github.com/notifications/unsubscribe-auth/AL07SsZUnZ6o2c-ESbgR8uPl-SreDvc-ks5tVaFLgaJpZM4QGVYe.

ny4i commented 6 years ago

I verified the default address works. I set my 7300 to use address 98h and I selected IC7610 in TR4W and it started communicating with the radio.

I will verify the commands sets again are identical for the functions we use.

n4af commented 6 years ago

Outstanding. ! Thanks Tom

On Wed, Mar 7, 2018 at 3:31 PM, Tom Schaefer notifications@github.com wrote:

I verified the default address works. I set my 7300 to use address 98h and I selected IC7610 in TR4W and it started communicating with the radio.

I will verify the commands sets again are identical for the functions we use.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-371274655, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqxUOWQYSG0w8PhmIWwm-ZP8MKs44RXks5tcEO-gaJpZM4QGVYe .

ny4i commented 6 years ago

I did find a change in the 7610 from the 7300 so I am checking that out now. There will be a commit for this.

Tom Principal Solutions Architect, ITIL®v3 Better Software Solutions, Inc. 727-437-BSS1 (727-437-2771) @BSSI_Consulting

On Mar 7, 2018, at 3:35 PM, Howie Hoyt notifications@github.com wrote:

Outstanding. ! Thanks Tom

On Wed, Mar 7, 2018 at 3:31 PM, Tom Schaefer notifications@github.com wrote:

I verified the default address works. I set my 7300 to use address 98h and I selected IC7610 in TR4W and it started communicating with the radio.

I will verify the commands sets again are identical for the functions we use.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-371274655, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqxUOWQYSG0w8PhmIWwm-ZP8MKs44RXks5tcEO-gaJpZM4QGVYe .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/n4af/TR4W/issues/282#issuecomment-371275547, or mute the thread https://github.com/notifications/unsubscribe-auth/AL07SuFY14MZqMLXN3SVgVDxIXbHnX-6ks5tcER2gaJpZM4QGVYe.

ny4i commented 4 years ago

This looks to be just like the 7300 except the extended mode command ($26) has a PKT mode and Multiple data modes. I wish I documented the difference I found so I will have to look closer. Extended Mode Commands 00: LSB 01: USB 02: AM 03: CW 04: RTTY 05: FM 07: CW-R 08: RTTY-R 12: PSK 08: PSK-R

00: Data mode OFF 01: Data mode 1 (D1) 02: Data mode 2 (D2) 03: Data mode 3 (D3)

What are the three Data modes?

01: FIL1 02: FIL2 03: FIL3

ny4i commented 2 years ago

This appears fixed. I asked a local friend with an IC-7610 to check the PSK mode reflects as DATA in the program.