search

cnr-ibf-pa / hbp-bsp-issues

Ticketing system for developers/testers and power users of the Brain Simulation Platform of the Human Brain Project
4 stars 0 forks source link

Add disclaimers for the use of the current hippocampus circuit #445

Closed romani79 closed 5 years ago

romani79 commented 5 years ago

Michele Migliore mentioned it should be clear to the user that the current version of the hippocampus model is there only for demo purposes and it should not be used to produce scientific results. Michele suggested to add some disclaimers before the users access the model.

antonelepfl commented 5 years ago

A really easy way to fix that is add some lines in the current Terms & Conditions that we show for the use cases. We can put something generic for all the models that are available on the BSP.

mmigliore commented 5 years ago

No, BSP models in general are supposed to be coming from publications! They can (and we would like them to) be used for scientific purposes. This message instead applies only for the models that are not yet (or will not be) published.

On Thu, Jun 20, 2019 at 6:17 AM Stefano Antonel notifications@github.com wrote:

A really easy way to fix that is add some lines in the current Terms & Conditions https://github.com/antonelepfl/usecases/blob/dev/src/assets/config_files/terms_and_conditions.md that we show for the use cases. We can put something generic for all the models that are available on the BSP.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cnr-ibf-pa/hbp-bsp-issues/issues/445?email_source=notifications&email_token=AGZUP3YGNXK66GR5UU4P4ETP3MAFJA5CNFSM4HZJ23Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYD6U6Y#issuecomment-503835259, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZUP34A6KLOYO6UX7LSRPTP3MAFJANCNFSM4HZJ23QQ .

antonelepfl commented 5 years ago

And if we add in the description of the Brain Area - Hippocampus that "this models are for demo ..." until we release the new version and we remove that description? Or we can add another 'pill' on the right of that use case that says 'demo' and when hover an explanation of it.

mmigliore commented 5 years ago

Better. Could you try something?

Il gio 20 giu 2019, 07:45 Stefano Antonel notifications@github.com ha scritto:

And if we add in the description of the Brain Area - Hippocampus https://collab.humanbrainproject.eu/#/collab/1655/nav/66856 that "this models are for demo ..." until we release the new version and we remove that description? Or we can add another 'pill' on the right of that use case that says 'demo' and when hover an explanation of it.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cnr-ibf-pa/hbp-bsp-issues/issues/445?email_source=notifications&email_token=AGZUP37EIODD5WIJB26IGQLP3MKQ7A5CNFSM4HZJ23Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYEKTQI#issuecomment-503884225, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZUP3ZSGVAPUDMOGYCCJQ3P3MKQ7ANCNFSM4HZJ23QQ .

stefanoantonel commented 5 years ago

So I have just dome some quick tests just to get an idea of what I was thinking. So basically we will add a new icon and when the user hover a tooltip will appear (as we have for the maturity level and the experience) Let me know what do you think and if there is an option that you like:

Screenshot 2019-06-21 at 14 02 02 Screenshot 2019-06-21 at 13 54 13 Screenshot 2019-06-21 at 13 55 58
alex4200 commented 5 years ago

I am in favor for the first option because it contains an exclamation mark. But maybe the text can be extended to "! DEMO MODEL" or "! DEMO CIRCUIT" so it is clear what the demo refers to.

stefanoantonel commented 5 years ago

Remember that when you hover a more descriptive information will appear

jdcourcol commented 5 years ago

I tend to agree with Alex. May be "Warning: model is work in progress"

mmigliore commented 5 years ago

I agree, with DEMO FULL CIRCUIT and a bigger triangle

Il ven 21 giu 2019, 14:21 alex4200 notifications@github.com ha scritto:

I am in favor for the first option because it contains an exclamation mark. But maybe the text can be extended to "! DEMO MODEL" or "! DEMO CIRCUIT" so it is clear what the demo refers to.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cnr-ibf-pa/hbp-bsp-issues/issues/445?email_source=notifications&email_token=AGZUP367ZVZ62NZRASFJPCDP3TBTTA5CNFSM4HZJ23Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYIJ6SY#issuecomment-504405835, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZUP35T2Y2CUO4IBCAJ6ETP3TBTTANCNFSM4HZJ23QQ .

antonelepfl commented 5 years ago

I have deployed this in the DEV BSP