El método volts_to_raw() veo perfectamente ponerlo oculto, sí. Ya lo cambio
yo ahora, porque estoy trabajando con daq.py para hacer la validación de
parámetros, así que ya hago ese cambio en la próxima subida.

Lo de la lógica de conversión, el problema es que esta función, según donde
se utilice, necesita hacerse después esa operación de multiplicar por 2 si
el modelo es el S o no, ya que volts_to_raw() también se utiliza, por
ejemplo, en set_analog(), pero en este caso no se debe multiplicar por 2 el
valor final, entonces no se podría incluir esa operación dentro de
volts_to_raw()


2014-04-10 11:34 GMT+02:00 Juan Menéndez <notifications@github.com>:

> In opendaq/daq.py:
>
> >
> >          Args:
> >              data: Total number of data points [1:400]
> >              offset: Offset for each value
> >          """
> > +        values = []
> > +        for volts in data:
> > +            raw = self.volts_to_raw(volts)
> > +            if self.hw_ver == "s":
>
> ¿No debería estar toda la lógica de conversión de valores en la función
> volts_to_raw? Eso incluye hacer lo que sea necesario según el modelo del
> hardware.
> Por otro lado, creo que el método volts_to_raw debería estar oculto
> (__volts_to_raw)
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/openDAQ/opendaq-python/pull/2/files#r11476656>
> .
>