AW: Driver PLC4J S7

2021-11-17 Thread Christofer Dutz
Hi Cesar,

it wasn't meant at all in any form that I was disapointed ... no worries ... I 
just wanted to encourage you that I am 1% sure I didn't get everything 
right (What would be the fun in that?). 

Chris


-Ursprüngliche Nachricht-
Von: Cesar Garcia  
Gesendet: Mittwoch, 17. November 2021 18:33
An: Apache PLC4X 
Betreff: Re: Driver PLC4J S7

Jeje, I don't understand the German sense of humor, but I've always said that 
the PLC4X team has done an excellent job, and it's always a pleasure to be able 
to help with something, it's just my grain of sand

El mié, 17 nov 2021 a las 13:03, Christofer Dutz ()
escribió:

> Cool ... didn't know we were doing things sub-ideal ... but hey ... 
> PLCs were new to me ;) Perhaps this can also help with the LOGO .. but 
> I would not expect it to help.
>
> Chris
>
> -Ursprüngliche Nachricht-
> Von: Cesar Garcia 
> Gesendet: Mittwoch, 17. November 2021 17:41
> An: Apache PLC4X 
> Betreff: Re: Driver PLC4J S7
>
> Hello Chris,
>
> Yes, I will verify, I know I saw the problem and just ignored it to 
> continue my tests. Given that we can now ask for the partial 
> diagnostic lists (SZL), it is easier to carry out tests, which is the 
> way in which the driver detects the type of device.
>
> Working on it, 
>
> El mié, 17 nov 2021 a las 12:28, Christofer Dutz (<
> christofer.d...@c-ware.de>)
> escribió:
>
> > HI Cesar,
> >
> > if you assign it in the connection string, it just skips the 
> > detection and sets the same variable to the same value as if you had 
> > auto-detected
> it.
> >
> > However ... if the auto detection is causing problems, perhaps this 
> > is something worth fixing. I know the LOGO devices just hang up if 
> > you ask them to identify themselves, but for the real S7 devices, we 
> > should make sure this works.
> >
> > Chris
> >
> >
> > -Ursprüngliche Nachricht-
> > Von: Cesar Garcia 
> > Gesendet: Mittwoch, 17. November 2021 15:37
> > An: Apache PLC4X 
> > Betreff: Re: Driver PLC4J S7
> >
> > Hello,
> >
> > Thanks for your comments,
> >
> > Yes, my target is those S7-300 and S7-400 computers, I forget the 
> > new neighbors in the neighborhood.
> >
> > As Chris points out, there is already the automatic detection of the 
> > PLC type, but in particular I prefer to assign it in the URL since 
> > in some scenario it fails, but I do not have it in my memory.
> >
> > I will test type detection with different equipment, add type 
> > discrimination for subscription, and document it so the point is 
> > clear to users.
> >
> > Best regards,
> >
> > El mié, 17 nov 2021 a las 5:47, Christofer Dutz (<
> > christofer.d...@c-ware.de>)
> > escribió:
> >
> > > Hi Lukasz,
> > >
> > > thanks for bringing this up ... I mentioned it on slack too.
> > > I am very happy with all of the recent extensions, but also we 
> > > already had first support requests from people trying to use the 
> > > subscription on non-compatible devices. So I would also like to 
> > > see the driver more context aware ... we already do the 
> > > identification as part of the connection process and should be 
> > > easily available to the driver implementation. If this 
> > > identification doesn't identify all types of devices (You 
> > > mentioned some non-s7 devices) I am happy to help with
> > extending the auto-detection.
> > >
> > > The S7DriverContext type has a controllerType enum value for this ...
> > > if the Enum S7ControllerType needs any addition, feel free to add them.
> > >
> > > Chris
> > >
> > >
> > >
> > > -Ursprüngliche Nachricht-
> > > Von: Łukasz Dywicki 
> > > Gesendet: Mittwoch, 17. November 2021 09:33
> > > An: dev@plc4x.apache.org
> > > Betreff: Re: Driver PLC4J S7
> > >
> > > Hey Cesar,
> > > I see that you do awesome progress with all complicated parts of 
> > > S7 communication. By looking at code and updated docs I see that 
> > > most of this functionality is intended for S7-400 and will not 
> > > work with low end S7-1200. This brings me to a question if our 
> > > present "PlcConnection.canSubscribe()" method is sufficient to let 
> > > know user application that it can subscribe? Does it return false 
> > > for S7-1200 and true for 300 and 400?
> > >
> > > Main question is how we could keep user applications aware of 
> > > subscription poss

Re: Driver PLC4J S7

2021-11-17 Thread Cesar Garcia
Jeje, I don't understand the German sense of humor, but I've always said
that the PLC4X team has done an excellent job, and it's always a pleasure
to be able to help with something, it's just my grain of sand

El mié, 17 nov 2021 a las 13:03, Christofer Dutz ()
escribió:

> Cool ... didn't know we were doing things sub-ideal ... but hey ... PLCs
> were new to me ;)
> Perhaps this can also help with the LOGO .. but I would not expect it to
> help.
>
> Chris
>
> -Ursprüngliche Nachricht-
> Von: Cesar Garcia 
> Gesendet: Mittwoch, 17. November 2021 17:41
> An: Apache PLC4X 
> Betreff: Re: Driver PLC4J S7
>
> Hello Chris,
>
> Yes, I will verify, I know I saw the problem and just ignored it to
> continue my tests. Given that we can now ask for the partial diagnostic
> lists (SZL), it is easier to carry out tests, which is the way in which the
> driver detects the type of device.
>
> Working on it, 
>
> El mié, 17 nov 2021 a las 12:28, Christofer Dutz (<
> christofer.d...@c-ware.de>)
> escribió:
>
> > HI Cesar,
> >
> > if you assign it in the connection string, it just skips the detection
> > and sets the same variable to the same value as if you had auto-detected
> it.
> >
> > However ... if the auto detection is causing problems, perhaps this is
> > something worth fixing. I know the LOGO devices just hang up if you
> > ask them to identify themselves, but for the real S7 devices, we
> > should make sure this works.
> >
> > Chris
> >
> >
> > -Ursprüngliche Nachricht-
> > Von: Cesar Garcia 
> > Gesendet: Mittwoch, 17. November 2021 15:37
> > An: Apache PLC4X 
> > Betreff: Re: Driver PLC4J S7
> >
> > Hello,
> >
> > Thanks for your comments,
> >
> > Yes, my target is those S7-300 and S7-400 computers, I forget the new
> > neighbors in the neighborhood.
> >
> > As Chris points out, there is already the automatic detection of the
> > PLC type, but in particular I prefer to assign it in the URL since in
> > some scenario it fails, but I do not have it in my memory.
> >
> > I will test type detection with different equipment, add type
> > discrimination for subscription, and document it so the point is clear
> > to users.
> >
> > Best regards,
> >
> > El mié, 17 nov 2021 a las 5:47, Christofer Dutz (<
> > christofer.d...@c-ware.de>)
> > escribió:
> >
> > > Hi Lukasz,
> > >
> > > thanks for bringing this up ... I mentioned it on slack too.
> > > I am very happy with all of the recent extensions, but also we
> > > already had first support requests from people trying to use the
> > > subscription on non-compatible devices. So I would also like to see
> > > the driver more context aware ... we already do the identification
> > > as part of the connection process and should be easily available to
> > > the driver implementation. If this identification doesn't identify
> > > all types of devices (You mentioned some non-s7 devices) I am happy
> > > to help with
> > extending the auto-detection.
> > >
> > > The S7DriverContext type has a controllerType enum value for this ...
> > > if the Enum S7ControllerType needs any addition, feel free to add them.
> > >
> > > Chris
> > >
> > >
> > >
> > > -Ursprüngliche Nachricht-
> > > Von: Łukasz Dywicki 
> > > Gesendet: Mittwoch, 17. November 2021 09:33
> > > An: dev@plc4x.apache.org
> > > Betreff: Re: Driver PLC4J S7
> > >
> > > Hey Cesar,
> > > I see that you do awesome progress with all complicated parts of S7
> > > communication. By looking at code and updated docs I see that most
> > > of this functionality is intended for S7-400 and will not work with
> > > low end S7-1200. This brings me to a question if our present
> > > "PlcConnection.canSubscribe()" method is sufficient to let know user
> > > application that it can subscribe? Does it return false for S7-1200
> > > and true for 300 and 400?
> > >
> > > Main question is how we could keep user applications aware of
> > > subscription possibilities without complicating PLC4X end too much.
> > > If subscription support is similar (alarm, mode, user) and available
> > > only in higher end models then we can live with current APIs, but if
> > > subscription support vary between models we need to think how to
> > > express that to end user. After all doing a subscription request
> > 

AW: Driver PLC4J S7

2021-11-17 Thread Christofer Dutz
Cool ... didn't know we were doing things sub-ideal ... but hey ... PLCs were 
new to me ;)
Perhaps this can also help with the LOGO .. but I would not expect it to help.

Chris

-Ursprüngliche Nachricht-
Von: Cesar Garcia  
Gesendet: Mittwoch, 17. November 2021 17:41
An: Apache PLC4X 
Betreff: Re: Driver PLC4J S7

Hello Chris,

Yes, I will verify, I know I saw the problem and just ignored it to continue my 
tests. Given that we can now ask for the partial diagnostic lists (SZL), it is 
easier to carry out tests, which is the way in which the driver detects the 
type of device.

Working on it, 

El mié, 17 nov 2021 a las 12:28, Christofer Dutz ()
escribió:

> HI Cesar,
>
> if you assign it in the connection string, it just skips the detection 
> and sets the same variable to the same value as if you had auto-detected it.
>
> However ... if the auto detection is causing problems, perhaps this is 
> something worth fixing. I know the LOGO devices just hang up if you 
> ask them to identify themselves, but for the real S7 devices, we 
> should make sure this works.
>
> Chris
>
>
> -Ursprüngliche Nachricht-
> Von: Cesar Garcia 
> Gesendet: Mittwoch, 17. November 2021 15:37
> An: Apache PLC4X 
> Betreff: Re: Driver PLC4J S7
>
> Hello,
>
> Thanks for your comments,
>
> Yes, my target is those S7-300 and S7-400 computers, I forget the new 
> neighbors in the neighborhood.
>
> As Chris points out, there is already the automatic detection of the 
> PLC type, but in particular I prefer to assign it in the URL since in 
> some scenario it fails, but I do not have it in my memory.
>
> I will test type detection with different equipment, add type 
> discrimination for subscription, and document it so the point is clear 
> to users.
>
> Best regards,
>
> El mié, 17 nov 2021 a las 5:47, Christofer Dutz (<
> christofer.d...@c-ware.de>)
> escribió:
>
> > Hi Lukasz,
> >
> > thanks for bringing this up ... I mentioned it on slack too.
> > I am very happy with all of the recent extensions, but also we 
> > already had first support requests from people trying to use the 
> > subscription on non-compatible devices. So I would also like to see 
> > the driver more context aware ... we already do the identification 
> > as part of the connection process and should be easily available to 
> > the driver implementation. If this identification doesn't identify 
> > all types of devices (You mentioned some non-s7 devices) I am happy 
> > to help with
> extending the auto-detection.
> >
> > The S7DriverContext type has a controllerType enum value for this ...
> > if the Enum S7ControllerType needs any addition, feel free to add them.
> >
> > Chris
> >
> >
> >
> > -Ursprüngliche Nachricht-
> > Von: Łukasz Dywicki 
> > Gesendet: Mittwoch, 17. November 2021 09:33
> > An: dev@plc4x.apache.org
> > Betreff: Re: Driver PLC4J S7
> >
> > Hey Cesar,
> > I see that you do awesome progress with all complicated parts of S7 
> > communication. By looking at code and updated docs I see that most 
> > of this functionality is intended for S7-400 and will not work with 
> > low end S7-1200. This brings me to a question if our present 
> > "PlcConnection.canSubscribe()" method is sufficient to let know user 
> > application that it can subscribe? Does it return false for S7-1200 
> > and true for 300 and 400?
> >
> > Main question is how we could keep user applications aware of 
> > subscription possibilities without complicating PLC4X end too much. 
> > If subscription support is similar (alarm, mode, user) and available 
> > only in higher end models then we can live with current APIs, but if 
> > subscription support vary between models we need to think how to 
> > express that to end user. After all doing a subscription request 
> > which fails is least effective way to find that driver and connected 
> > plc does
> not support specific subscription kind.
> >
> > Best,
> > Łukasz
> >
> > śr., 17 lis 2021 o 06:43 Cesar Garcia 
> > napisał(a):
> >
> > > Hello,
> > >
> > > This PR includes the following features for the S7 driver.
> > >
> > > 1. Subscribe to Alarm8.
> > >2. Recognition of events.
> > >3. Alarm text processing.
> > >4. Subscription to cyclical values.
> > >5. Diagnostic Record Readings (SZL).
> > >6. Proposal for documentation.
> > >
> > > Thank you very much for your support in the documentation 
> > > proposal, several sections are missing, but 

Re: Driver PLC4J S7

2021-11-17 Thread Cesar Garcia
Hello Chris,

Yes, I will verify, I know I saw the problem and just ignored it to
continue my tests. Given that we can now ask for the partial diagnostic
lists (SZL), it is easier to carry out tests, which is the way in which the
driver detects the type of device.

Working on it, 

El mié, 17 nov 2021 a las 12:28, Christofer Dutz ()
escribió:

> HI Cesar,
>
> if you assign it in the connection string, it just skips the detection and
> sets the same variable to the same value as if you had auto-detected it.
>
> However ... if the auto detection is causing problems, perhaps this is
> something worth fixing. I know the LOGO devices just hang up if you ask
> them to identify themselves, but for the real S7 devices, we should make
> sure this works.
>
> Chris
>
>
> -Ursprüngliche Nachricht-
> Von: Cesar Garcia 
> Gesendet: Mittwoch, 17. November 2021 15:37
> An: Apache PLC4X 
> Betreff: Re: Driver PLC4J S7
>
> Hello,
>
> Thanks for your comments,
>
> Yes, my target is those S7-300 and S7-400 computers, I forget the new
> neighbors in the neighborhood.
>
> As Chris points out, there is already the automatic detection of the PLC
> type, but in particular I prefer to assign it in the URL since in some
> scenario it fails, but I do not have it in my memory.
>
> I will test type detection with different equipment, add type
> discrimination for subscription, and document it so the point is clear to
> users.
>
> Best regards,
>
> El mié, 17 nov 2021 a las 5:47, Christofer Dutz (<
> christofer.d...@c-ware.de>)
> escribió:
>
> > Hi Lukasz,
> >
> > thanks for bringing this up ... I mentioned it on slack too.
> > I am very happy with all of the recent extensions, but also we already
> > had first support requests from people trying to use the subscription
> > on non-compatible devices. So I would also like to see the driver more
> > context aware ... we already do the identification as part of the
> > connection process and should be easily available to the driver
> > implementation. If this identification doesn't identify all types of
> > devices (You mentioned some non-s7 devices) I am happy to help with
> extending the auto-detection.
> >
> > The S7DriverContext type has a controllerType enum value for this ...
> > if the Enum S7ControllerType needs any addition, feel free to add them.
> >
> > Chris
> >
> >
> >
> > -Ursprüngliche Nachricht-
> > Von: Łukasz Dywicki 
> > Gesendet: Mittwoch, 17. November 2021 09:33
> > An: dev@plc4x.apache.org
> > Betreff: Re: Driver PLC4J S7
> >
> > Hey Cesar,
> > I see that you do awesome progress with all complicated parts of S7
> > communication. By looking at code and updated docs I see that most of
> > this functionality is intended for S7-400 and will not work with low
> > end S7-1200. This brings me to a question if our present
> > "PlcConnection.canSubscribe()" method is sufficient to let know user
> > application that it can subscribe? Does it return false for S7-1200
> > and true for 300 and 400?
> >
> > Main question is how we could keep user applications aware of
> > subscription possibilities without complicating PLC4X end too much. If
> > subscription support is similar (alarm, mode, user) and available only
> > in higher end models then we can live with current APIs, but if
> > subscription support vary between models we need to think how to
> > express that to end user. After all doing a subscription request which
> > fails is least effective way to find that driver and connected plc does
> not support specific subscription kind.
> >
> > Best,
> > Łukasz
> >
> > śr., 17 lis 2021 o 06:43 Cesar Garcia 
> > napisał(a):
> >
> > > Hello,
> > >
> > > This PR includes the following features for the S7 driver.
> > >
> > > 1. Subscribe to Alarm8.
> > >2. Recognition of events.
> > >3. Alarm text processing.
> > >4. Subscription to cyclical values.
> > >5. Diagnostic Record Readings (SZL).
> > >6. Proposal for documentation.
> > >
> > > Thank you very much for your support in the documentation proposal,
> > > several sections are missing, but I think your observations are
> > > extremely important.
> > >
> > > Best regards,
> > >
> > > --
> > > *CEOS Automatización, C.A.*
> > > *GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,* *PISO 1, OFICINA
> > > 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*
> > >
> > > *FRENTE A LA ASOCI

AW: Driver PLC4J S7

2021-11-17 Thread Christofer Dutz
HI Cesar,

if you assign it in the connection string, it just skips the detection and sets 
the same variable to the same value as if you had auto-detected it. 

However ... if the auto detection is causing problems, perhaps this is 
something worth fixing. I know the LOGO devices just hang up if you ask them to 
identify themselves, but for the real S7 devices, we should make sure this 
works.

Chris


-Ursprüngliche Nachricht-
Von: Cesar Garcia  
Gesendet: Mittwoch, 17. November 2021 15:37
An: Apache PLC4X 
Betreff: Re: Driver PLC4J S7

Hello,

Thanks for your comments,

Yes, my target is those S7-300 and S7-400 computers, I forget the new neighbors 
in the neighborhood.

As Chris points out, there is already the automatic detection of the PLC type, 
but in particular I prefer to assign it in the URL since in some scenario it 
fails, but I do not have it in my memory.

I will test type detection with different equipment, add type discrimination 
for subscription, and document it so the point is clear to users.

Best regards,

El mié, 17 nov 2021 a las 5:47, Christofer Dutz ()
escribió:

> Hi Lukasz,
>
> thanks for bringing this up ... I mentioned it on slack too.
> I am very happy with all of the recent extensions, but also we already 
> had first support requests from people trying to use the subscription 
> on non-compatible devices. So I would also like to see the driver more 
> context aware ... we already do the identification as part of the 
> connection process and should be easily available to the driver 
> implementation. If this identification doesn't identify all types of 
> devices (You mentioned some non-s7 devices) I am happy to help with extending 
> the auto-detection.
>
> The S7DriverContext type has a controllerType enum value for this ... 
> if the Enum S7ControllerType needs any addition, feel free to add them.
>
> Chris
>
>
>
> -Ursprüngliche Nachricht-
> Von: Łukasz Dywicki 
> Gesendet: Mittwoch, 17. November 2021 09:33
> An: dev@plc4x.apache.org
> Betreff: Re: Driver PLC4J S7
>
> Hey Cesar,
> I see that you do awesome progress with all complicated parts of S7 
> communication. By looking at code and updated docs I see that most of 
> this functionality is intended for S7-400 and will not work with low 
> end S7-1200. This brings me to a question if our present 
> "PlcConnection.canSubscribe()" method is sufficient to let know user 
> application that it can subscribe? Does it return false for S7-1200 
> and true for 300 and 400?
>
> Main question is how we could keep user applications aware of 
> subscription possibilities without complicating PLC4X end too much. If 
> subscription support is similar (alarm, mode, user) and available only 
> in higher end models then we can live with current APIs, but if 
> subscription support vary between models we need to think how to 
> express that to end user. After all doing a subscription request which 
> fails is least effective way to find that driver and connected plc does not 
> support specific subscription kind.
>
> Best,
> Łukasz
>
> śr., 17 lis 2021 o 06:43 Cesar Garcia 
> napisał(a):
>
> > Hello,
> >
> > This PR includes the following features for the S7 driver.
> >
> > 1. Subscribe to Alarm8.
> >2. Recognition of events.
> >3. Alarm text processing.
> >4. Subscription to cyclical values.
> >5. Diagnostic Record Readings (SZL).
> >6. Proposal for documentation.
> >
> > Thank you very much for your support in the documentation proposal, 
> > several sections are missing, but I think your observations are 
> > extremely important.
> >
> > Best regards,
> >
> > --
> > *CEOS Automatización, C.A.*
> > *GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,* *PISO 1, OFICINA 
> > 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*
> >
> > *FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI* *Ing.
> > César García*
> >
> > *Cel: +58 414-760.98.95*
> >
> > *Hotline Técnica SIEMENS: 0800 1005080*
> >
> > *Email: support.aan.automat...@siemens.com
> > *
> >
>


--
*CEOS Automatización, C.A.*
*GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,* *PISO 1, OFICINA 2, AV. RAUL 
LEONI, SECTOR GUAMACHITO,*

*FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI* *Ing. César 
García*

*Cel: +58 414-760.98.95*

*Hotline Técnica SIEMENS: 0800 1005080*

*Email: support.aan.automat...@siemens.com
*


Re: Driver PLC4J S7

2021-11-17 Thread Cesar Garcia
Hello,

Thanks for your comments,

Yes, my target is those S7-300 and S7-400 computers, I forget the new
neighbors in the neighborhood.

As Chris points out, there is already the automatic detection of the PLC
type, but in particular I prefer to assign it in the URL since in some
scenario it fails, but I do not have it in my memory.

I will test type detection with different equipment, add type
discrimination for subscription, and document it so the point is clear to
users.

Best regards,

El mié, 17 nov 2021 a las 5:47, Christofer Dutz ()
escribió:

> Hi Lukasz,
>
> thanks for bringing this up ... I mentioned it on slack too.
> I am very happy with all of the recent extensions, but also we already had
> first support requests from people trying to use the subscription on
> non-compatible devices. So I would also like to see the driver more context
> aware ... we already do the identification as part of the connection
> process and should be easily available to the driver implementation. If
> this identification doesn't identify all types of devices (You mentioned
> some non-s7 devices) I am happy to help with extending the auto-detection.
>
> The S7DriverContext type has a controllerType enum value for this ... if
> the Enum S7ControllerType needs any addition, feel free to add them.
>
> Chris
>
>
>
> -Ursprüngliche Nachricht-
> Von: Łukasz Dywicki 
> Gesendet: Mittwoch, 17. November 2021 09:33
> An: dev@plc4x.apache.org
> Betreff: Re: Driver PLC4J S7
>
> Hey Cesar,
> I see that you do awesome progress with all complicated parts of S7
> communication. By looking at code and updated docs I see that most of this
> functionality is intended for S7-400 and will not work with low end
> S7-1200. This brings me to a question if our present
> "PlcConnection.canSubscribe()" method is sufficient to let know user
> application that it can subscribe? Does it return false for S7-1200 and
> true for 300 and 400?
>
> Main question is how we could keep user applications aware of subscription
> possibilities without complicating PLC4X end too much. If subscription
> support is similar (alarm, mode, user) and available only in higher end
> models then we can live with current APIs, but if subscription support vary
> between models we need to think how to express that to end user. After all
> doing a subscription request which fails is least effective way to find
> that driver and connected plc does not support specific subscription kind.
>
> Best,
> Łukasz
>
> śr., 17 lis 2021 o 06:43 Cesar Garcia 
> napisał(a):
>
> > Hello,
> >
> > This PR includes the following features for the S7 driver.
> >
> > 1. Subscribe to Alarm8.
> >2. Recognition of events.
> >3. Alarm text processing.
> >4. Subscription to cyclical values.
> >5. Diagnostic Record Readings (SZL).
> >6. Proposal for documentation.
> >
> > Thank you very much for your support in the documentation proposal,
> > several sections are missing, but I think your observations are
> > extremely important.
> >
> > Best regards,
> >
> > --
> > *CEOS Automatización, C.A.*
> > *GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,* *PISO 1, OFICINA 2,
> > AV. RAUL LEONI, SECTOR GUAMACHITO,*
> >
> > *FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI* *Ing.
> > César García*
> >
> > *Cel: +58 414-760.98.95*
> >
> > *Hotline Técnica SIEMENS: 0800 1005080*
> >
> > *Email: support.aan.automat...@siemens.com
> > *
> >
>


-- 
*CEOS Automatización, C.A.*
*GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,*
*PISO 1, OFICINA 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*

*FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI*
*Ing. César García*

*Cel: +58 414-760.98.95*

*Hotline Técnica SIEMENS: 0800 1005080*

*Email: support.aan.automat...@siemens.com
*


AW: Driver PLC4J S7

2021-11-17 Thread Christofer Dutz
Hi Lukasz,

thanks for bringing this up ... I mentioned it on slack too.
I am very happy with all of the recent extensions, but also we already had 
first support requests from people trying to use the subscription on 
non-compatible devices. So I would also like to see the driver more context 
aware ... we already do the identification as part of the connection process 
and should be easily available to the driver implementation. If this 
identification doesn't identify all types of devices (You mentioned some non-s7 
devices) I am happy to help with extending the auto-detection. 

The S7DriverContext type has a controllerType enum value for this ... if the 
Enum S7ControllerType needs any addition, feel free to add them.

Chris



-Ursprüngliche Nachricht-
Von: Łukasz Dywicki  
Gesendet: Mittwoch, 17. November 2021 09:33
An: dev@plc4x.apache.org
Betreff: Re: Driver PLC4J S7

Hey Cesar,
I see that you do awesome progress with all complicated parts of S7 
communication. By looking at code and updated docs I see that most of this 
functionality is intended for S7-400 and will not work with low end S7-1200. 
This brings me to a question if our present "PlcConnection.canSubscribe()" 
method is sufficient to let know user application that it can subscribe? Does 
it return false for S7-1200 and true for 300 and 400?

Main question is how we could keep user applications aware of subscription 
possibilities without complicating PLC4X end too much. If subscription support 
is similar (alarm, mode, user) and available only in higher end models then we 
can live with current APIs, but if subscription support vary between models we 
need to think how to express that to end user. After all doing a subscription 
request which fails is least effective way to find that driver and connected 
plc does not support specific subscription kind.

Best,
Łukasz

śr., 17 lis 2021 o 06:43 Cesar Garcia  napisał(a):

> Hello,
>
> This PR includes the following features for the S7 driver.
>
> 1. Subscribe to Alarm8.
>2. Recognition of events.
>3. Alarm text processing.
>4. Subscription to cyclical values.
>5. Diagnostic Record Readings (SZL).
>6. Proposal for documentation.
>
> Thank you very much for your support in the documentation proposal, 
> several sections are missing, but I think your observations are 
> extremely important.
>
> Best regards,
>
> --
> *CEOS Automatización, C.A.*
> *GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,* *PISO 1, OFICINA 2, 
> AV. RAUL LEONI, SECTOR GUAMACHITO,*
>
> *FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI* *Ing. 
> César García*
>
> *Cel: +58 414-760.98.95*
>
> *Hotline Técnica SIEMENS: 0800 1005080*
>
> *Email: support.aan.automat...@siemens.com
> *
>


Re: Driver PLC4J S7

2021-11-17 Thread Łukasz Dywicki
Hey Cesar,
I see that you do awesome progress with all complicated parts of S7
communication. By looking at code and updated docs I see that most of this
functionality is intended for S7-400 and will not work with low end
S7-1200. This brings me to a question if our present
"PlcConnection.canSubscribe()" method is sufficient to let know user
application that it can subscribe? Does it return false for S7-1200 and
true for 300 and 400?

Main question is how we could keep user applications aware of subscription
possibilities without complicating PLC4X end too much. If subscription
support is similar (alarm, mode, user) and available only in higher end
models then we can live with current APIs, but if subscription support vary
between models we need to think how to express that to end user. After all
doing a subscription request which fails is least effective way to find
that driver and connected plc does not support specific subscription kind.

Best,
Łukasz

śr., 17 lis 2021 o 06:43 Cesar Garcia  napisał(a):

> Hello,
>
> This PR includes the following features for the S7 driver.
>
> 1. Subscribe to Alarm8.
>2. Recognition of events.
>3. Alarm text processing.
>4. Subscription to cyclical values.
>5. Diagnostic Record Readings (SZL).
>6. Proposal for documentation.
>
> Thank you very much for your support in the documentation proposal, several
> sections are missing, but I think your observations are extremely
> important.
>
> Best regards,
>
> --
> *CEOS Automatización, C.A.*
> *GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,*
> *PISO 1, OFICINA 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*
>
> *FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI*
> *Ing. César García*
>
> *Cel: +58 414-760.98.95*
>
> *Hotline Técnica SIEMENS: 0800 1005080*
>
> *Email: support.aan.automat...@siemens.com
> *
>


Driver PLC4J S7

2021-11-16 Thread Cesar Garcia
Hello,

This PR includes the following features for the S7 driver.

1. Subscribe to Alarm8.
   2. Recognition of events.
   3. Alarm text processing.
   4. Subscription to cyclical values.
   5. Diagnostic Record Readings (SZL).
   6. Proposal for documentation.

Thank you very much for your support in the documentation proposal, several
sections are missing, but I think your observations are extremely important.

Best regards,

-- 
*CEOS Automatización, C.A.*
*GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,*
*PISO 1, OFICINA 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*

*FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI*
*Ing. César García*

*Cel: +58 414-760.98.95*

*Hotline Técnica SIEMENS: 0800 1005080*

*Email: support.aan.automat...@siemens.com
*