Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Två buggar (och fulfixar) #1

Open
hakber opened this issue Sep 20, 2021 · 2 comments
Open

Två buggar (och fulfixar) #1

hakber opened this issue Sep 20, 2021 · 2 comments

Comments

@hakber
Copy link

hakber commented Sep 20, 2021

Ber om ursäkt, har dålig koll på github-ettikett och hur man brukar göra, men jag stötte på två buggar i alla fall och har gjort några fulfixar själv. Jag har bara tankat grundskoledata ännu:

I utils.py på rad 110 så avsågs att returnera (float) [...]. Jag tror det blir konstigt för vissa värden i databasen på grundskolesidan är skrivna som "~100" och liknande. Jag ändrade den till (string), så funkar det temporärt (så kan man kanske konvertera datatyper själv i ett senare skede, lite efter behov).

Jag fick det vidare inte att funka med periods-queryt, oavsett om jag skrev typ "2014", "2014/2015", ["2014"], ["2014/2015"], [("2014", "2014/2015")] eller någon annan snarlik permutation. Men om jag kommenterade bort rad 96 i scraper.py så gick det i alla fall att kalla med ["2014"] och motsvarande.

Tack för ett sjukt värdefullt script, underlättar massa för mig!

@rotsee
Copy link
Member

rotsee commented Sep 22, 2021

Stort tack för buggrapporten! Vi får uppdatera nästa gång vi använder skriptet (eller om du vill göra en pull request med dina ändringar, så kan vi dra in den direkt). Det där med ~100 tror jag är ganska nytt (antar att det är statistiksekretess som slår till där?)

@hakber
Copy link
Author

hakber commented Sep 24, 2021

Kul! Tack för svar. Jag ska ta och dubbelkolla det bara så att det inte är jag som kodade fel eller så, men jag gör ett försök på en pull request sedan om ni inte hinner före :).
Kan vara statistiksekretess angående ~100, låter rimligt!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants