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

Backend refactors #232

Merged
merged 8 commits into from
Dec 10, 2023
Merged

Backend refactors #232

merged 8 commits into from
Dec 10, 2023

Conversation

jannisborn
Copy link
Owner

INFO:main:======Location differ joblog from 2023-12-09 22:30:04=======
INFO:googlemaps.client:API queries_quota: 60
INFO:main:To-be-added-machine Insectarium in Pennsylvania seems unavailable: http://209.221.138.252/Details.aspx?location=204156 with Internal Server Error (500)
INFO:main:To-be-added-machine True Grits Gift & Cards in Georgia seems unavailable: http://209.221.138.252/Details.aspx?location=1860 with Internal Server Error (500)
INFO:main:1/40: Found machine to be added: Unknown Location
ERROR:pennyme.pennycollector:Geolocation failed for: Unknown Location sub: , Arco
INFO:main:To-be-added-machine Sunderland Football Stadium in England seems unavailable: http://209.221.138.252/Details.aspx?location=4284 with Internal Server Error (500)
INFO:main:To-be-added-machine Six Flags Mexico in Mexico seems unavailable: http://209.221.138.252/Details.aspx?location=713 with Internal Server Error (500)
INFO:main:
Result: 0 changes, 0 new machines found and 0 machines retired
ERROR:main:Found 5 problems that require manual intervention
INFO:main:======Location differ completed at 2023-12-10 00:23:48=======

@jannisborn jannisborn added bot PR created by a bot data Machine location updates labels Dec 9, 2023
@jannisborn jannisborn changed the title Updates from website (problems json) Minor backend refactors Dec 10, 2023
@jannisborn
Copy link
Owner Author

  • Adapted location-differ to be more silent......
  • decided to increase the image width from 400 to 1000 pixels when processing uploaded images (currently most images are very pixelated)
  • disentangled two machines in NOLA museum

@jannisborn
Copy link
Owner Author

Dont merge this yet, I will add here the location-differ refactor that uses the new logic for machine statuses

@jannisborn jannisborn marked this pull request as ready for review December 10, 2023 21:15
@jannisborn jannisborn changed the title Minor backend refactors Backend refactors Dec 10, 2023
@NinaWie NinaWie merged commit c812b85 into main Dec 10, 2023
1 check passed
@NinaWie NinaWie deleted the machine_updates branch December 10, 2023 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot PR created by a bot data Machine location updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants