raise ValueError("data_service_url must end in '.org'")

[hazaioud]

Hello,

I want to use this code to download data from pvoutput.org. I don't have any PV system registered but I've donated in order to be able to download the data of other pv systems. I installed the library via the pip install method. After that importing the library is not causing any problem.

The problem occurs when I want to acces a PV system. I do this by writing 'PVOutput(API_KEY, SYSTEM_ID)'. With API_Key= my API_key that i can find under settings in pvouput.org; and System_ID= the id of the system from which I want to acces the data, so the 5 numbers coming after &sid= in the url. This gives me following error in python:

File "", line 1, in
PVOutput(API_KEY, SYSTEM_ID)

File "c:\users\hazaioud\src\pvoutput\pvoutput\pvoutput.py", line 91, in init
raise ValueError("data_service_url must end in '.org'")

ValueError: data_service_url must end in '.org'

After that I've putted source code as comment:

if self.data_service_url is not None:

if not self.data_service_url.strip('/').endswith('.org'):

raise ValueError("data_service_url must end in '.org'")

Than I don't see the error anymore but I'm not sure wether that is intented or not. Anyway, after that I try to asses the pvsystems within a radius of 5 km from the given coordinates:

pv.search(query='5km', lat=52.0668589, lon=-1.3484038)

But than Python freezes totally.....

What I'm doing wrong?

[JackKelly]

Hi @hazaioud sorry to hear the code isn't working correctly. I'm afraid I'm really busy with a number of other projects, so I'm not sure when I'll next get a chance to look at this code... hopefully in the not-too-distant-future...

[ssmssam]

Hi @hazaioud, out of interest were you trying to do this on a Windows machine? I have created a pull request, so hopefully this will work for you soon if you are still interested in the data.

[peterdudfield]

Hi @hazaioud. I was just browsing this issue. Do you know if this is solved?

p.s thanks @ssmssam for those changes.

[hazaioud]

Hello, @ssmssam<https://github.com/ssmssam> notified me that this should work now, thanks for that. Unfortunately I did not found the time yet to check it. But as I’m trying it on a Windows machine I expect it will work well. Kind regards Hakim Azaioud From: Peter Dudfield ***@***.***> Sent: dinsdag 4 januari 2022 13:57 To: openclimatefix/pvoutput ***@***.***> Cc: Hakim Azaioud ***@***.***>; Mention ***@***.***> Subject: Re: [openclimatefix/pvoutput] raise ValueError("data_service_url must end in '.org'") (#38) Hi @hazaioud<https://github.com/hazaioud>. I was just browsing this issue. Do you know if this is solved? p.s thanks @ssmssam<https://github.com/ssmssam> for those changes. — Reply to this email directly, view it on GitHub<#38 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AQBIUBEOTWVNTYT4DHC7UC3UULVDJANCNFSM4OPE4F6A>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>