Skip to content
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

Feature/plotter widget #448 #452

Merged
merged 15 commits into from
Feb 4, 2019
Merged

Conversation

blondejamtart
Copy link
Contributor

Description

  • Added new widget for plotting array or table attributes
  • Many tweaks/minor bug fixes raised by user testing
  • Added formatting and units to textInput and textUpdate for numerical attributes
  • Fixed bug with deleting read-only links from layouts

Testing instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check Travis build
  • Review changes to test coverage

Agile board tracking

connect to #448
connect to #322
connect to #447
connect to #450
connect to #321
connect to #6
closes #448
closes #322
closes #447
closes #450
closes #321
closes #6

Bryan Robert Tester added 7 commits January 17, 2019 12:09
…+ other fixes

- Layout view button now displays a graphic which looks suitably layout-y
- TextInput/Update widgets now format their content + display units based on the display_t from the provided meta
- Fixed some widget sizes when displayed in attributeDetails
- Added fix to prevent layout attempting to delete/change read-only sink port attributes
Bryan Robert Tester added 2 commits January 31, 2019 15:29
- Fixed plot widget for arrays
- Added option of number of points to plot
- Improved layout optimistic link delete behaviour
- Added code to allow for running auto-layout on local state only
- Renamed display_t to display to account for corresponding change in pymalcolm
@codecov-io
Copy link

codecov-io commented Jan 31, 2019

Codecov Report

Merging #452 into master will decrease coverage by 2.59%.
The diff coverage is 41.93%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #452     +/-   ##
=========================================
- Coverage   82.57%   79.97%   -2.6%     
=========================================
  Files          81       85      +4     
  Lines        3334     3511    +177     
  Branches      615      656     +41     
=========================================
+ Hits         2753     2808     +55     
- Misses        512      601     +89     
- Partials       69      102     +33
Impacted Files Coverage Δ
src/malcolm/reducer/method.reducer.js 96.7% <ø> (ø) ⬆️
...dlePanelViews/methodView/methodViewer.component.js 75% <ø> (ø) ⬆️
src/layout/layout.component.js 60.46% <ø> (ø) ⬆️
src/malcolmWidgets/method/method.component.js 88.88% <ø> (+3.55%) ⬆️
src/malcolmWidgets/checkbox/checkbox.component.js 100% <ø> (ø) ⬆️
...lmWidgets/textUpdate/WidgetTextUpdate.component.js 100% <ø> (ø) ⬆️
src/middlePanelViews/connectIFrame.component.js 0% <0%> (ø)
src/mainMalcolmView/middlePanel.container.js 70.21% <0%> (-1.53%) ⬇️
src/malcolmWidgets/table/table.container.js 91.48% <0%> (ø) ⬆️
...rc/middlePanelViews/tabbedMiddlePanel.component.js 23.07% <0%> (+1.64%) ⬆️
... and 26 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84a2341...d1e8a3d. Read the comment docs.

@blondejamtart blondejamtart merged commit b2c3b95 into master Feb 4, 2019
@blondejamtart blondejamtart deleted the feature/plotterWidget-#448 branch February 4, 2019 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants