ssc-oscar / DRE

MIT License
4 stars 10 forks source link

Updates and fixes #37

Closed sylviesworld closed 4 years ago

sylviesworld commented 4 years ago
audrism commented 4 years ago

Awesome!

audrism commented 4 years ago
sylviesworld commented 4 years ago

I used the latest implementation of fastgraph that was in master at the time. I didn't make a link to it because it wasn't working yet -- It is trying to do mappings using version R which I think is why it isn't working. Once we get it working I can make a link to it in the toolbar.

Here is the error that fastgraph gives: (node:9466) UnhandledPromiseRejectionWarning: Error: Command failed: ssh -T da4 << EOF echo "42103c6630f9e026ac9d5aa53753803b68247c18" | $HOME/lookup/getValues c2P EOF ls: cannot access /da?_data/basemaps/c2PFullR.31.tch: No such file or directory

branson2015 commented 4 years ago

I had to add a bash script to swsc/lookup repo that was not included in the DRE PR. It was basically a one-liner wrapper around the perl script - I could probably remove this, but the existing code used this same architecture, so I just followed suit. This is probably why it does not work if you go to it directly by the url, but, rest assured, I do have a version of fastgraph working with the latest perl script additions.

On Thu, Nov 5, 2020 at 11:11 AM Dylan Lomax notifications@github.com wrote:

I used the latest implementation of fastgraph that was in master at the time. I didn't make a link to it because it wasn't working yet -- It is trying to do mappings using version R which I think is why it isn't working. Once we get it working I can make a link to it in the toolbar.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ssc-oscar/DRE/pull/37#issuecomment-722476077, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACZ4DZKHFYYSOGK4MWEJEN3SOLFDRANCNFSM4TK22MAQ .

audrism commented 4 years ago

Great I merged your the PR.

Is there a way to send objects as standard input, however. For example, getNeighborhood author name might be very hard to parse from the command line?

So it would be easier if it was provided as standard input

audrism commented 4 years ago

Ok, I added author, project, file, tree, blob search as the parameter types a, p, f, tree, blob