bird-team / brisbane-bird-atlas

Atlas of the Birds of Brisbane: Community bird atlas for Brisbane, Australia
https://brisbanebirds.com
GNU General Public License v3.0
3 stars 0 forks source link

Trouble remaking assets #153

Closed Louis-Backstrom closed 3 years ago

Louis-Backstrom commented 3 years ago

Trying to update Atlas assets to the latest version of eBird data (Dec-2020). Docker seems to be playing up:

I have the latest versions of Cygwin, Docker, and Rtools installed and running, everything seems to be in order but when I run make pull_image I get the following:

make: docker: Command not found
make: *** [Makefile:228: pull_image] Error 127

Google searching doesn't give me particularly informative solutions to the 127 error.

Thoughts?

jeffreyhanson commented 3 years ago

Ok, can you please pull the latest commit and run make test_access?

jeffreyhanson commented 3 years ago

You should see something like this in the output:

PING www.google.com (172.217.25.36) 56(84) bytes of data.
64 bytes from syd15s02-in-f36.1e100.net (172.217.25.36): icmp_seq=1 ttl=113 time=266 ms
64 bytes from syd15s02-in-f36.1e100.net (172.217.25.36): icmp_seq=2 ttl=113 time=192 ms
64 bytes from syd15s02-in-f36.1e100.net (172.217.25.36): icmp_seq=3 ttl=113 time=153 ms

--- www.google.com ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2001ms
jeffreyhanson commented 3 years ago

Assuming that it has internet connection

Louis-Backstrom commented 3 years ago
PING www.google.com (142.250.66.196) 56(84) bytes of data.
64 bytes from 142.250.66.196 (142.250.66.196): icmp_seq=1 ttl=37 time=26.2 ms
64 bytes from 142.250.66.196 (142.250.66.196): icmp_seq=2 ttl=37 time=23.5 ms
64 bytes from 142.250.66.196 (142.250.66.196): icmp_seq=3 ttl=37 time=27.4 ms

--- www.google.com ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2003ms
rtt min/avg/max/mdev = 23.517/25.749/27.437/1.651 ms

Looks fine??

jeffreyhanson commented 3 years ago

Yeah - seems like it can indeed access the internet

jeffreyhanson commented 3 years ago

Hmm, I've added some debugging info to the code for pushing the assets. Can you pull the latest version and run make push_assets? This will print a bunch of stuff, including the token. Could you please verify that the token is correct, and paste the results here (replacing the token with XXX)?

jeffreyhanson commented 3 years ago

Just to clarify, it should print the token twice.

Louis-Backstrom commented 3 years ago
> piggyback::pb_info(bird-team/brisbane-bird-atlas, v.0.0.1)
Error: 'pb_info' is not an exported object from 'namespace:piggyback'
Execution halted
&& docker exec bba sh -c "R -e 'f=tempfile();writeLines(\"this is a test2\", f);piggyback::pb_upload(file=f, repo=\"bird-team/brisbane-bird-atlas\", tag=\"v.0.0.1\", name=\"new-test5.txt\")'" || true
sh: -c: line 0: syntax error near unexpected token `&&'
sh: -c: line 0: `&& docker exec bba sh -c "R -e 'f=tempfile();writeLines(\"this is a test2\", f);piggyback::pb_upload(file=f, repo=\"bird-team/brisbane-bird-atlas\", tag=\"v.0.0.1\", name=\"new-test5.txt\")'" || true'
make: *** [Makefile:143: test_access] Error 1

Think you've got a typo?

jeffreyhanson commented 3 years ago

Yeah, sorry, there was a typo - can you try again?

Louis-Backstrom commented 3 years ago
> piggyback:::pb_info(bird-team/brisbane-bird-atlas, v.0.0.1)
Error in FUN(X[[i]], ...) : object 'bird' not found
Calls: <Anonymous> -> memoised_pb_info -> lapply -> FUN -> FUN
Execution halted
&& docker exec bba sh -c "R -e 'f=tempfile();writeLines(\"this is a test2\", f);piggyback::pb_upload(file=f, repo=\"bird-team/brisbane-bird-atlas\", tag=\"v.0.0.1\", name=\"new-test5.txt\")'" || true
sh: -c: line 0: syntax error near unexpected token `&&'
sh: -c: line 0: `&& docker exec bba sh -c "R -e 'f=tempfile();writeLines(\"this is a test2\", f);piggyback::pb_upload(file=f, repo=\"bird-team/brisbane-bird-atlas\", tag=\"v.0.0.1\", name=\"new-test5.txt\")'" || true'
make: *** [Makefile:143: test_access] Error 1

Hmm

Louis-Backstrom commented 3 years ago

@jeffreyhanson ?

jeffreyhanson commented 3 years ago

Ah, sorry for the slow response. That's my fault - due to a typo. Would you be free sometime this week to try and sort this out over Skype and TeamViewer (so I can remotely connect to your computer and try stuff without having to bother you over GitHub)? If today suits, I'm free later this afternoon?

Louis-Backstrom commented 3 years ago

Sorry it took a while to get to this - I could do tonight, or otherwise anytime tomorrow should be fine?

jeffreyhanson commented 3 years ago

Ok, how does 3PM tomorrow NZ time (12PM tomorrow Brisbane time) work?

Louis-Backstrom commented 3 years ago

Sounds excellent - lock it in.

jeffreyhanson commented 3 years ago

Ok great - see you then

Louis-Backstrom commented 3 years ago

Loading required package: methods
Linking to GEOS 3.5.1, GDAL 2.1.2, proj.4 4.9.3
Quitting from lines 28-29 (brisbane-bird-atlas.Rmd) 
Error in unzip(dir("data/records", "^.*\\.zip$", full.names = TRUE), exdir = tmp1) : 
  invalid zip name argument
Calls: system.time ... eval -> source -> withVisible -> eval -> eval -> unzip

Timing stopped at: 7.532 0.176 7.714
Execution halted

Exited with code exit status 1
CircleCI received exit code 1

@jeffreyhanson

jeffreyhanson commented 3 years ago

Thanks for the heads up - I've just pushed some (attempted) fixes.

jeffreyhanson commented 3 years ago

Looks like both the Travis and CircleCI builds just finished - so I think everything is working again?

Louis-Backstrom commented 3 years ago

Huzzah! Guess we can finally close this issue now :)