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

Update Spark job for https://github.com/archivesunleashed/aut/pull/289 #211

Closed
ruebot opened this issue Nov 21, 2018 · 5 comments

Comments

@ruebot
Copy link
Member

commented Nov 21, 2018

When the next aut release happens, we'll need to update the Spark background job with the new syntax.

@ianmilligan1 ianmilligan1 self-assigned this Nov 21, 2018
@ianmilligan1

This comment has been minimized.

Copy link
Member

commented Nov 22, 2018

Now that this pull request is merged into AUT, I'm going to do some local tests on AUK w/ the new syntax and can start a branch that can be ultimately merged once 0.18.0 is released.

@greebie

This comment has been minimized.

Copy link
Contributor

commented Nov 22, 2018

Oh yes. I said I would test Graphpass results. Will do that this afternoon.

@greebie

This comment has been minimized.

Copy link
Contributor

commented Nov 22, 2018

image

Basic graphpass run ran as expected (I expected it to). Ids turned out great! Don't forget to use WriteGraph.asGraphml(file) for the script.

@ianmilligan1

This comment has been minimized.

Copy link
Member

commented Nov 28, 2018

You mean WriteGraph.asGraphml(file) not ExtractGraph, right @greebie? The new PR dealt with the former.

@greebie

This comment has been minimized.

Copy link
Contributor

commented Nov 28, 2018

Yes. I updated the comment. Sorry about that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.