MiguelAngelLV / gas_station_spain

Home Assistant Component for Gas Stations of Spain
MIT License
20 stars 6 forks source link

Sugerencias de mejora #1

Closed tergontor closed 8 months ago

tergontor commented 8 months ago

Probablemente puede ser problema de la API pero por si acaso comento algún defecto que he podido encontrar.
1- De las gasolineras que sigo hay una que no la encuentro en la integración. El código que tengo puesto y que funciona es

- scan_interval: 1800
    resource: https://sedeaplicaciones.minetur.gob.es/ServiciosRESTCarburantes/PreciosCarburantes/EstacionesTerrestres/FiltroMunicipio/3782
    sensor:
      - name: "Gasol.95 Easyoil Pto Manzanal km 351"
        value_template: >-
          {% set station = value_json['ListaEESSPrecio'] | selectattr('IDEESS', 'match', '4937') | first %}
          {{ station['Precio Gasolina 95 E5'] | replace(",",".") }}
        unit_of_measurement: "€/L"

Lo he buscado en la provincia de León pero no encuentro el pueblo de Manzanal del Puerto.

2- Había una gasolinera que por código daba los datos pero por la integración sacaba "desconocido". Y digo había, porque en la actualidad ya marca el precio correcto. Se trataba de la gasolinera situada en Madrid, Cepsa, calle Monte Igueldo.

3- Esto es una sugerencia de mejora en el caso que se pueda. Se trata de que automáticamente el nombre de la entidad fuera el que aparece en el listado de la gasolinera, así no hay lugar a confusión si añades diferentes gasolineras de la misma marca. Luego puedes cambiar el nombre de la entidad si así se desea.

Saludos y gracias.

MiguelAngelLV commented 8 months ago

1): La gasolinera que has puesto, la 4937, corresponde:

        {
            "C.P.": "24369",
            "Dirección": "CARRETERA N - VI KM. 351",
            "Horario": "L-D: 24H",
            "Latitud": "42,599222",
            "Localidad": "MANZANAL DEL PUERTO",
            "Longitud (WGS84)": "-6,230444",
            "Margen": "I",
            "Municipio": "Villagatón",
            "Precio Biodiesel": "",
            "Precio Bioetanol": "",
            "Precio Gas Natural Comprimido": "",
            "Precio Gas Natural Licuado": "",
            "Precio Gases licuados del petróleo": "",
            "Precio Gasoleo A": "1,569",
            "Precio Gasoleo B": "",
            "Precio Gasoleo Premium": "1,569",
            "Precio Gasolina 95 E10": "",
            "Precio Gasolina 95 E5": "1,679",
            "Precio Gasolina 95 E5 Premium": "1,679",
            "Precio Gasolina 98 E10": "",
            "Precio Gasolina 98 E5": "",
            "Precio Hidrogeno": "",
            "Provincia": "LEÓN",
            "Remisión": "dm",
            "Rótulo": "EASYGAS",
            "Tipo Venta": "P",
            "% BioEtanol": "0,0",
            "% Éster metílico": "0,0",
            "IDEESS": "4937",
            "IDMunicipio": "3782",
            "IDProvincia": "24",
            "IDCCAA": "08"
        }

No debes buscar por localidad, sino por municipio, está en «Villagatón».

2) Seguramente en el momento de intentar hacer la consulta, pillaría el servicio caído, y como hace consultas cada varias horas, hasta que no ha vuelto a tocar consultar pues se ha quedado con el error.

3) Entiendo que te refieres que, además del municipio y el rótulo, se añada también la dirección?

Actualmente es así: Gasolina 95 E , Easygas Entiendo que quieres así: Gasolina 95 E, Easygas Carretera N-Vi km351

Si es eso, poder se puede, pero la mayoría de las direcciones suelen ser muy largas y tengo que ver los tamaños máximos admitidos.

tergontor commented 8 months ago

OK, me obcequé con lo del Puerto del Manzanal y pensaba que era la localidad. Respecto al nombre del sensor, con poner Gas. 95 ( o diesel) marca, municipio y algo que diferencie ( por ejemplo en Madrid habría que añadir algo más) creo que sería suficiente. Luego si queda largo siempre se puede renombrar pero de esta manera sabes perfectamente a qué gasolinera te estás refiriendo. Un saludo y muchas gracias de nuevo.

tergontor commented 8 months ago

Y puestos a pedir ya que hablas de coordenadas y viendo el video de Tecnoyfoto. Ya que las coordenadas de cada gasolinera vienen en el código de la api ¿se podría desarrollar para que te sacara dependiendo de tus coordenadas las gasolineras que están más baratas en un radio de x km? Sería muy util cuando estás de viaje pero supongo que será bastante complicado interactuar con la ubicación que se tenga en ese momento. Pero por pedir que no quede. Es lo que tiene ser un absoluto negado en la programación. Saludos,

MiguelAngelLV commented 8 months ago

Tengo pensado añadir las coordenadas, pero el sacar las más baratas por radio sería inviable gracias a la chapuza que hizo la empresa contratada por el estada.

No ofrece ningún tipo de búsqueda (ni por coordenadas ni nada). Tendría que descargar todas las gasolineras y buscar, y lógicamente, el coste computacional es bastante grande.

Es más, ni siquiera ofrece una forma de obtener directamente una gasolinera, tengo que sacar todas las del municipio y filtrar después...

tergontor commented 8 months ago

Entendido, gracias Miguel Ángel, de chapuzas de empresas contratadas por el Estado, o por grandes compañías, se podría escribir un libro. Saludos y gracias por la respuesta.

MiguelAngelLV commented 8 months ago

He subido una versión en beta, ahora al añadirlo además de del tipo y el rótulo, añade el nombre de la calle y el número.

Échale un ojo.

tergontor commented 8 months ago

Hola Miguel Angel, disculpa por no contestarte antes pero he estado de viaje y no he regresado hasta hoy. He visto que ya hay una nueva actualización con importantes mejoras. Así que enhorabuena y muchas gracias por todo. Saludos, Javier

El sáb, 14 oct 2023 a las 8:05, Miguel Ángel López Vicente (< @.***>) escribió:

He subido una versión en beta, ahora al añadirlo además de del tipo y el rótulo, añade el nombre de la calle y el número.

Échale un ojo.

— Reply to this email directly, view it on GitHub https://github.com/MiguelAngelLV/gas_station_spain/issues/1#issuecomment-1762636515, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPQ6CZWZCVR7HQ7JGO7PELX7ITTBANCNFSM6AAAAAA5XW5C4Y . You are receiving this because you authored the thread.Message ID: @.***>

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Libre de virus.www.avast.com https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>