Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Histogram and stats#1030

Merged
dhaneshnm merged 17 commits into1.7.0-M4from
3006-histo
Jan 31, 2022
Merged

Histogram and stats#1030
dhaneshnm merged 17 commits into1.7.0-M4from
3006-histo

Conversation

@smitfire
Copy link
Contributor

@smitfire smitfire commented Jan 21, 2022 •
edited
Loading

Fixes #
Screenshot 2022-01-21 at 13 29 53
Screenshot 2022-01-21 at 13 29 49

Description

How has this been tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [ x] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added necessary unit and integration tests.
  • [ x] I have added screenshots (if applicable), in the comment section.

nicwells and others added 5 commits January 18, 2022 15:57
@nicwells
3012 Fix paging when querying Elasticsearch views (#1025)
e7afaad 
* Prevent losing change to size when updating from

* Upgrade nexus-sdk package to version which fixes paging

* Shorthand properties first
@smitfire
stats added
e2da18c
@smitfire
added cahrt lib and two options for the histos
edd11de
@smitfire
added checkbox for histo
893ae3c
@smitfire
formatting
1b3e1cf
@smitfire smitfire mentioned this pull request Jan 21, 2022
Closed
@codecov-commenter
Copy link

codecov-commenter commented Jan 21, 2022 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (1.7.0-M4@78e1455). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head ad26056 differs from pull request most recent head d74604e. Consider uploading reports for the commit d74604e to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##             1.7.0-M4    #1030   +/-   ##
===========================================
  Coverage            ?   66.61%           
===========================================
  Files               ?       22           
  Lines               ?      668           
  Branches            ?      150           
===========================================
  Hits                ?      445           
  Misses              ?      223           
  Partials            ?        0

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 78e1455...d74604e. Read the comment docs.

@smitfire smitfire changed the base branch from main to 1.7.0-M4 January 21, 2022 13:07
@smitfire
dynamic histo inteval
c288581
@smitfire
assigning min correctly
28a6ac2
@smitfire
removed g
8c6ed1b
@smitfire
fixed charting bug
d879ab1
@smitfire
Merge branch '3006-histo' of https://github.com/BlueBrain/nexus-web i…
e510ad4 
…nto 3006-histo

* '3006-histo' of https://github.com/BlueBrain/nexus-web:
  fixed charting bug
  removed g
  assigning min correctly
  dynamic histo inteval
  proper reference to stats
  formatting
  added checkbox for histo
  added cahrt lib and two options for the histos
  stats added
@smitfire
fixed bouton density intreval calc
45a1a13
@smitfire
formatting
04978a2
@smitfire
changed the interval condition from greater than 0 to 1
5e26cf9
@nicwells
Copy link
Contributor

nicwells commented Jan 26, 2022

just noticed build is failing here due to running out of memory

@dhaneshnm dhaneshnm merged commit d27aa86 into 1.7.0-M4 Jan 31, 2022
@dhaneshnm dhaneshnm deleted the 3006-histo branch January 31, 2022 10:48
dhaneshnm pushed a commit that referenced this pull request Mar 11, 2022
@smitfire @nicwells @dhaneshnm
Histogram and stats (#1030)
8c47174 
* 3012 Fix paging when querying Elasticsearch views (#1025)
* split react, antd, antv in to smaller chunks (#1036)


Co-authored-by: Nick Wells <wells.n@gmail.com>
Co-authored-by: Dhanesh Neela Mana <dhanesh.neelamana@epfl.ch>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@dhaneshnm dhaneshnm dhaneshnm approved these changes

@nicwells nicwells nicwells approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@smitfire @codecov-commenter @nicwells @dhaneshnm