expose convert_edgelist (#56)

[schochastics]

Turned it into a generic function that works with rang and ranglet objects.

[chainsawriot]

@schochastics If you prefer

el <- do.call("rbind",lapply(x$ranglets,convert_edgelist))

can be replaced by

el <- vctrs::vec_rbind(lapply(x$ranglets,convert_edgelist))

now, probably slightly faster and more robust.

[codecov-commenter]

Codecov Report

Merging #74 (17afdfa) into v0.2 (fabb92d) will increase coverage by 0.98%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             v0.2      #74      +/-   ##
==========================================
+ Coverage   92.51%   93.49%   +0.98%     
==========================================
  Files           5        6       +1     
  Lines         641      646       +5     
==========================================
+ Hits          593      604      +11     
+ Misses         48       42       -6     

Impacted Files	Coverage Δ
R/resolve.R	90.03% <ø> (+1.81%)	⬆️
R/edgelist.R	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[schochastics]

ah nice will do that

[schochastics]

hmm that didnt work
vctrs::vec_rbind doesnt seem to support a list of data frames as input?
@chainsawriot should I revert, or did I do something wrong?

[chainsawriot]

@schochastics Sorry, I checked and it doesn't work as I said. Please revert.

[schochastics]

@chainsawriot done. Happy that my faithful companion do.call for so many years remains useful

[chainsawriot]

Well

### but need to import the triple bang
vctrs::vec_rbind(!!!(lapply(graph$ranglets, convert_edgelist)))

Rather than being smart like this, I rather opt for being familiar. Please keep the do.call.

[schochastics]

haha I was actually wondering if a series of "!" could solve this issue. Well, TIL