dandi / example-notebooks

https://www.dandiarchive.org/example-notebooks
Apache License 2.0
6 stars 17 forks source link

Update and extend (OPEN vs EMBARGOED; more data types and species mapped) stats for DANDI #71

Closed yarikoptic closed 1 month ago

yarikoptic commented 7 months ago

Here are previews of some plots

image

image

yarikoptic commented 7 months ago

apparently codespell isn't happy -- will send fixes in a separate PR

bendichter commented 7 months ago

This is now really only meaningful if the user that runs it has admin privileges. For example, if I run this I only get 4 embargoed dandisets, since I am only an owner of 4. Given that, I think I'd rather this be in a separate place, distinct from the DANDI summary commands that can be run by any user.

yarikoptic commented 7 months ago

Well, I didn't realize this would be of interest to a regular user.

"separate place" == "duplication" == divergence of implementations etc. So I would really hope that we could avoid that.

Let me offer two solutions in the order of my preference ;)

WDYT @bendichter ?

NB thinking out loud... ideally we should just make this notebook run e.g. weekly and place those images somewhere on the website for up-to-date informatio

satra commented 7 months ago

i may have posted an issue to generate this dashboard directly on top of django, i.e. by our backend :)

yarikoptic commented 6 months ago

ok, I have added explicit embargoed = True and a comment so anon users could flip it if desired. Verified that it still works.

I rebased and reran so it is uptodate ATM. I addressed typo directly in master, rebased again...

@bendichter -- are you ok with this? (@satra already blessed PR)

yarikoptic commented 6 months ago

here is how that cell looks like now image

yarikoptic commented 1 month ago

oh, it was approved! Let's proceed!