Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 11:16
    brianhelba synchronize #3132
  • 11:16

    brianhelba on ansible-doc

    Update documentation for Ansibl… (compare)

  • 11:13
    brianhelba synchronize #3132
  • 11:13

    brianhelba on ansible-doc

    Update documentation for Ansibl… (compare)

  • 11:12
    brianhelba opened #3132
  • 11:12
    brianhelba review_requested #3132
  • 11:12
    brianhelba review_requested #3132
  • 11:11

    brianhelba on ansible-doc

    Update documentation for Ansibl… (compare)

  • 07:39
    brianhelba commented #3131
  • 07:37
    brianhelba commented #3131
  • 07:04
    brianhelba commented #3131
  • 06:53
    brianhelba commented #3131
  • 06:46
    mgrauer opened #3131
  • Oct 10 15:02
    zachmullen closed #3130
  • Oct 10 14:37
    zachmullen commented #3130
  • Oct 10 14:32
    sgratzl edited #3130
  • Oct 10 14:32
    sgratzl opened #3130
  • Oct 08 18:47
    zachmullen labeled #3129
  • Oct 08 18:46
    zachmullen labeled #3129
  • Oct 08 18:42
    sgratzl commented #3129
Kacper Kowalik
@Xarthisius
is that more or less secure than passing cookie?
Zach Mullen
@zachmullen
roughly the same, but then the third party client is responsible for reading it from the redirect URL and storing it to use as the token for future girder requests
which you'd probably do with a cookie that you'd use just to persist the data
Zach Mullen
@zachmullen
that's probably the way to go. After reading more about explicit domain cookies, it seems that it will limit you to only doing CORS across subdomains
Kacper Kowalik
@Xarthisius
you mean the ?token approach ?
Zach Mullen
@zachmullen
yeah
here is what I was reading
we should probably just do a bit of extra logic such that if the specified redirect value contains some special string like $GIRDER_TOKEN, that gets replaced with the actual token when we perform the redirect
Andrey Fedorov
@fedorov
I installed girder with pip. How do I upgrade to the latest version?
Zach Mullen
@zachmullen
pip install -U girder
Andrey Fedorov
@fedorov
thanks!
Zach Mullen
@zachmullen
sure thing :)
Andrey Fedorov
@fedorov
hmm, not so simple. After doing that, and restarting girder-server, I am getting this error while trying to access the web page: TypeError: 'BaseWebroot' object is not callable
Jonathan Beezley
@jbeezley
You probably need to run girder-install web to rebuild the client code.
Andrey Fedorov
@fedorov
blob
another problem there ...
Jonathan Beezley
@jbeezley
We require npm >=3. Can you npm install -g npm (may require sudo)?
Andrey Fedorov
@fedorov
blob
another problem next
Zach Mullen
@zachmullen
ah, I have seen something like this when upgrading from girder 1.x to 2.x
You may need to blow away the node_modules directory inside the girder package
Jonathan Beezley
@jbeezley
Yeah, I think it has to do with updating from npm 2 to 3
the resolution semantics changed
Zach Mullen
@zachmullen
could also work to just pip uninstall girder && pip install girder :P
Jonathan Owens
@jonathan-owens
I’m trying to extend the CheckedMenuWidget to display an additional action available when at least one item is selected.
I can get the widget to update its rendering using girder.wrap(girder.views.CheckedMenuWidget …
However, when I try to bind an event to clicking the new item with girder.views.CheckedMenuWidget.prototype.events, I get an “Undefined is not an object"
I did a little digging, and it seems that CheckedMenuWidget has no events field…so that could be a hint
But I’m new to JavaScript and don’t really no where to go from here.
Any tips?
Andrey Fedorov
@fedorov
@zachmullen @jbeezley pip uninstall girder && pip install girder did not help, but removing node_modules and re-running girder-install web did the job, thank you! Would it make sense to have some documentation section with upgrade instructions?
Zach Mullen
@zachmullen
That's probably a good thing to put into the Migration Guide
Andrey Fedorov
@fedorov
@zachmullen I am happy to make a PR but not sure what are your conventions. How about adding the following paragraph in the beginning: "Existing installation may be upgraded by running pip install -U girder. You may need to remove node_modules directory from the installed girder package, and re-run girder-install web if you encounter problems. Note that the prerequisites may have changed in the latest version: make sure to review http://girder.readthedocs.io/en/latest/prerequisites.html prior to the upgrade."
Zach Mullen
@zachmullen
sounds great :)
Andrey Fedorov
@fedorov
girder/girder#1740
Jonathan Beezley
@jbeezley
@jonathan-owens If you’re wrapping an existing class, the way to add events is to wrap the initialize method and mutate this.events in the wrapper function.
Kacper Kowalik
@Xarthisius
hi! when I visit plugins' setting page I get Uncaught ReferenceError: webpackJsonp is not defined at plugin.min.js:1 with current HEAD
hmm it's gone after I cleaned cache
Kacper Kowalik
@Xarthisius
@zachmullen is girder.api.docs.addModel something waiting to be used or some old, forgotten code?
Kacper Kowalik
@Xarthisius
oh looks like PR 1701 is extremely relevant to what I'm doing right now :0
Zach Mullen
@zachmullen
cool :)
@cjh1 and I have been working on that, he knows more about the models than I do actually
but to my knowledge it does work
Hans Meine
@hmeine
@brianhelba gave a nice Girder demo last week at the NAMIC Project Week – I was wondering whether some of the installations he demo'ed (or similar ones) are publicly available. For instance, https://github.com/girder/monkeybrains/ does not link to any installation yet?
Dan LaManna
@danlamanna
@manthey weren't there demos of girder running on python2/3 somewhere? do they still exist?
Zach Mullen
@zachmullen
@hmeine the monkey brains plugin is deployed here: https://data.kitware.com/#collection/54b582c38d777f4362aa9cb3
Zach Mullen
@zachmullen
the COVALIC site is here: https://challenge.kitware.com/
the ISBI challenge in particular is here: https://challenge.kitware.com/#challenge/560d7856cad3a57cfde481ba
Jonathan Owens
@jonathan-owens
@jbeezley Thanks for the tip!