search

esmero / strawberry_runners

A post processing Drupal 8/9 module for Strawberryfield dispatched events
GNU Lesser General Public License v3.0
3 stars 2 forks source link

ISSUE-91:ML processors #92

Closed DiegoPino closed 2 months ago

DiegoPino commented 4 months ago

See #91

Fully depends on https://github.com/esmero/strawberryfield/pull/326

DiegoPino commented 2 months ago

@alliomeria I would like this is mentioned in the new release but documentation is something that we actually need to work internally and generate/decide on the demo implementations. The least harmful ML method here is mobileNet, so maybe enable as an admin that one? Also. I need to know which permission/users should be able to use ML driven Views for testing.

alliomeria commented 2 months ago

Ok @DiegoPino, understood about your preferences for this. Admin only is what you've been saying for awhile and sounds good to me too.

DiegoPino commented 2 months ago

@alliomeria the only issue with Admin Only is if we have a group of people working/testing (this is query) part that the institution does not trust on full admin access, but still provides Authenticated/User access. What if I make it "has a certain permissions" OR "is admin" but is not (never) "anonymous" ?

alliomeria commented 2 months ago

Would defer to you on this @DiegoPino, all depends on how widely you think this tool should be used. Having the option for any type of Authenticated User with the checked permissions, but never anonymous/non-authenticated users also works & enables non-admistrator level users to work/test as you noted.

DiegoPino commented 2 months ago

@alliomeria I will try to work tonight on generating a very simple documentation piece. Added the permissions and restrictions. This requires a new NLP processor container which I will build and publish tomorrow AM (takes me 3 hours of docker built at least). Thanks for reading all this, this is at least 3K lines of code