-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
include custom widget #149
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Ben Dichter <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #149 +/- ##
==========================================
- Coverage 69.15% 68.91% -0.25%
==========================================
Files 30 30
Lines 2169 2184 +15
==========================================
+ Hits 1500 1505 +5
- Misses 669 679 +10
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@luiztauffer see edits to my comment. You may have to test it a bit. I'm not 100% sure it will work as I wrote it |
Now that I am thinking about it, maybe the values of |
@bendichter I agree, at least with most of the items. It makes adding custom widgets easy |
This allows user-created widgets to be included in the
neurodata_vis_spec
, can be used like this:For now it only works for items which are
OrderedDict
. Maybe we would like to extend most items to beOrderedDict
? This would in principle just nest each item in a single Tab, but allows for the possibility of easily adding customized widgets