Skip to content

Commit

Permalink
Merge pull request #1684 from CentreForDigitalHumanities/bugfix/frequ…
Browse files Browse the repository at this point in the history
…ency-visualisation-int-fields

set minimum size for terms aggregation in visualisation
  • Loading branch information
lukavdplas authored Nov 6, 2024
2 parents 37ba05a + 315d2ee commit 2f5e77b
Showing 1 changed file with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,19 +41,16 @@ export class HistogramComponent
* used in document requests.
*/
getAggregator(): TermsAggregator {
let size = 0;

if (!this.visualizedField.filterOptions) {
return new TermsAggregator(this.visualizedField, 100);
}
let size = 100;

const filterOptions = this.visualizedField.filterOptions;
if (filterOptions.name === 'MultipleChoiceFilter') {
size = (filterOptions as MultipleChoiceFilterOptions).option_count;
} else if (filterOptions.name === 'RangeFilter') {
size =
const filterRange =
(filterOptions as RangeFilterOptions).upper -
(filterOptions as RangeFilterOptions).lower;
size = _.max([size, filterRange])
}
return new TermsAggregator(this.visualizedField, size);
}
Expand Down

0 comments on commit 2f5e77b

Please sign in to comment.