client error: (422) Unprocessable Entity (WebDAV; RFC 4918)

[cpsievert]

Sorry this isn't really a minimal example, but I'm not sure of a good way to reduce it:

library(git2r)
library(gistr)
repo <- clone("https://github.com/cpsievert/AnimintTest.git", "AnimintTest")
fs <- dir("AnimintTest/FluView", full.names = TRUE)
gist_create(fs, description = "FluView")

#> Error in list2gist(x) : Not coerceable to a gist
#> In addition: Warning message:
#>   In process(response) :
#>   client error: (422) Unprocessable Entity (WebDAV; RFC 4918)

[sckott]

@cpsievert you have a folder in that directory, which i think is what causes the problem.

[sckott]

@cpsievert what would you want to have happen with a folder ? e.g., recursive behavior so that all files in a folder are pushed up? Or if a folder is found, skip it?

[cpsievert]

whoops, sorry about that, maybe you could add a check at the start of gist_create() like:

if (any(file_test("-d", files))) stop("Can't upload directories as a gist.)

?

[sckott]

Yeah, that sounds good to stop if a folder is found,

other than this issue, I think this should work even with the hundreds of files you have in your e.g.

[cpsievert]

Yea, I forgot that in animint2dir() we flatten the file structure (which requires changing some hrefs). I'll play around some more and report back.

[sckott]

thanks 👍

[sckott]

added checks to gist_create() for binary files and directories in ad40063

[cpsievert]

Thanks, I think it's safe to close this.

[sckott]

cool, thanks for bringing this up

[cpsievert]

No prob, I flattened the file structure and now I get this error:

Error: Validation Failed
resource: Gist
code: missing_field
field: files
In addition: Warning message:
In max(b) : no non-missing arguments to max; returning -Inf

Perhaps I've hit the limit on the number of files?

Shall we migrate to another issue?

[sckott]

i'll try it again, and see if it's a diff. problem... 🕐

[sckott]

@cpsievert that warning In max(b) : no non-missing arguments to max; returning -Inf was just related to a helper fxn for checking for binary files. should be fixed now in commit not yet pushed

For the large number of files you have I think using git will be much easier and faster. Have you tried that? Is it do-able for your use case?

Using gist_create() with this example, it's failing, but haven't quite tracked down exactly why yet, something with the Github API

[cpsievert]

Got it, thanks, I'll think about using git2r in animint instead (I didn't know it was on CRAN!)

[sckott]

yeah, so nice that it's on cran now