Skip to content

Commit

Permalink
fix none label on distribution chart
Browse files Browse the repository at this point in the history
  • Loading branch information
DominikB2014 committed Oct 16, 2024
1 parent ba4be16 commit f6a4b24
Show file tree
Hide file tree
Showing 4 changed files with 77 additions and 41 deletions.
29 changes: 19 additions & 10 deletions static/app/views/insights/pages/ai/aiOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,10 @@ import {ProjectPageFilter} from 'sentry/components/organizations/projectPageFilt
import TransactionNameSearchBar from 'sentry/components/performance/searchBar';
import SentryDocumentTitle from 'sentry/components/sentryDocumentTitle';
import {trackAnalytics} from 'sentry/utils/analytics';
import {canUseMetricsData} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {
canUseMetricsData,
useMEPSettingContext,
} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {PageAlert, usePageAlert} from 'sentry/utils/performance/contexts/pageAlert';
import {PerformanceDisplayProvider} from 'sentry/utils/performance/contexts/performanceDisplayContext';
import {MutableSearch} from 'sentry/utils/tokenizeSearch';
Expand All @@ -28,6 +31,7 @@ import {AI_LANDING_TITLE} from 'sentry/views/insights/pages/ai/settings';
import {OVERVIEW_PAGE_TITLE} from 'sentry/views/insights/pages/settings';
import {generateGenericPerformanceEventView} from 'sentry/views/performance/data';
import {TripleChartRow} from 'sentry/views/performance/landing/widgets/components/widgetChartRow';
import {filterAllowedChartsMetrics} from 'sentry/views/performance/landing/widgets/utils';
import {PerformanceWidgetSetting} from 'sentry/views/performance/landing/widgets/widgetDefinitions';
import Onboarding from 'sentry/views/performance/onboarding';
import Table from 'sentry/views/performance/table';
Expand All @@ -54,6 +58,7 @@ function AiOverviewPage() {
const {projects} = useProjects();
const onboardingProject = useOnboardingProject();
const navigate = useNavigate();
const mepSetting = useMEPSettingContext();

const withStaticFilters = canUseMetricsData(organization);
const eventView = generateGenericPerformanceEventView(
Expand All @@ -76,15 +81,19 @@ function AiOverviewPage() {

const showOnboarding = onboardingProject !== undefined;

const tripleChartRowCharts = [
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
];
const tripleChartRowCharts = filterAllowedChartsMetrics(
organization,
[
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
],
mepSetting
);

const sharedProps = {eventView, location, organization, withStaticFilters};

Expand Down
31 changes: 20 additions & 11 deletions static/app/views/insights/pages/backend/backendOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,10 @@ import {ProjectPageFilter} from 'sentry/components/organizations/projectPageFilt
import TransactionNameSearchBar from 'sentry/components/performance/searchBar';
import SentryDocumentTitle from 'sentry/components/sentryDocumentTitle';
import {trackAnalytics} from 'sentry/utils/analytics';
import {canUseMetricsData} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {
canUseMetricsData,
useMEPSettingContext,
} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {PageAlert, usePageAlert} from 'sentry/utils/performance/contexts/pageAlert';
import {PerformanceDisplayProvider} from 'sentry/utils/performance/contexts/performanceDisplayContext';
import {MutableSearch} from 'sentry/utils/tokenizeSearch';
Expand All @@ -31,6 +34,7 @@ import {
DoubleChartRow,
TripleChartRow,
} from 'sentry/views/performance/landing/widgets/components/widgetChartRow';
import {filterAllowedChartsMetrics} from 'sentry/views/performance/landing/widgets/utils';
import {PerformanceWidgetSetting} from 'sentry/views/performance/landing/widgets/widgetDefinitions';
import Onboarding from 'sentry/views/performance/onboarding';
import Table from 'sentry/views/performance/table';
Expand Down Expand Up @@ -60,6 +64,7 @@ function BackendOverviewPage() {
const {projects} = useProjects();
const onboardingProject = useOnboardingProject();
const navigate = useNavigate();
const mepSetting = useMEPSettingContext();

const withStaticFilters = canUseMetricsData(organization);
const eventView = generateBackendPerformanceEventView(
Expand Down Expand Up @@ -91,16 +96,20 @@ function BackendOverviewPage() {
PerformanceWidgetSetting.SLOW_HTTP_OPS,
PerformanceWidgetSetting.SLOW_DB_OPS,
];
const tripleChartRowCharts = [
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
PerformanceWidgetSetting.APDEX_AREA,
];
const tripleChartRowCharts = filterAllowedChartsMetrics(
organization,
[
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
PerformanceWidgetSetting.APDEX_AREA,
],
mepSetting
);

if (organization.features.includes('insights-initial-modules')) {
doubleChartRowCharts.unshift(
Expand Down
29 changes: 19 additions & 10 deletions static/app/views/insights/pages/frontend/frontendOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,10 @@ import {ProjectPageFilter} from 'sentry/components/organizations/projectPageFilt
import TransactionNameSearchBar from 'sentry/components/performance/searchBar';
import SentryDocumentTitle from 'sentry/components/sentryDocumentTitle';
import {trackAnalytics} from 'sentry/utils/analytics';
import {canUseMetricsData} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {
canUseMetricsData,
useMEPSettingContext,
} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {PageAlert, usePageAlert} from 'sentry/utils/performance/contexts/pageAlert';
import {PerformanceDisplayProvider} from 'sentry/utils/performance/contexts/performanceDisplayContext';
import {MutableSearch} from 'sentry/utils/tokenizeSearch';
Expand All @@ -32,6 +35,7 @@ import {
DoubleChartRow,
TripleChartRow,
} from 'sentry/views/performance/landing/widgets/components/widgetChartRow';
import {filterAllowedChartsMetrics} from 'sentry/views/performance/landing/widgets/utils';
import {PerformanceWidgetSetting} from 'sentry/views/performance/landing/widgets/widgetDefinitions';
import Onboarding from 'sentry/views/performance/onboarding';
import Table from 'sentry/views/performance/table';
Expand Down Expand Up @@ -59,6 +63,7 @@ function FrontendOverviewPage() {
const {projects} = useProjects();
const onboardingProject = useOnboardingProject();
const navigate = useNavigate();
const mepSetting = useMEPSettingContext();

const withStaticFilters = canUseMetricsData(organization);
const eventView = generateFrontendOtherPerformanceEventView(
Expand Down Expand Up @@ -94,15 +99,19 @@ function FrontendOverviewPage() {
PerformanceWidgetSetting.SLOW_HTTP_OPS,
PerformanceWidgetSetting.SLOW_RESOURCE_OPS,
];
const tripleChartRowCharts = [
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
];
const tripleChartRowCharts = filterAllowedChartsMetrics(
organization,
[
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
],
mepSetting
);

if (organization.features.includes('insights-initial-modules')) {
doubleChartRowCharts.unshift(PerformanceWidgetSetting.MOST_TIME_CONSUMING_DOMAINS);
Expand Down
29 changes: 19 additions & 10 deletions static/app/views/insights/pages/mobile/mobileOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,10 @@ import {ProjectPageFilter} from 'sentry/components/organizations/projectPageFilt
import TransactionNameSearchBar from 'sentry/components/performance/searchBar';
import SentryDocumentTitle from 'sentry/components/sentryDocumentTitle';
import {trackAnalytics} from 'sentry/utils/analytics';
import {canUseMetricsData} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {
canUseMetricsData,
useMEPSettingContext,
} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
import {PageAlert, usePageAlert} from 'sentry/utils/performance/contexts/pageAlert';
import {PerformanceDisplayProvider} from 'sentry/utils/performance/contexts/performanceDisplayContext';
import {MutableSearch} from 'sentry/utils/tokenizeSearch';
Expand All @@ -35,6 +38,7 @@ import {
DoubleChartRow,
TripleChartRow,
} from 'sentry/views/performance/landing/widgets/components/widgetChartRow';
import {filterAllowedChartsMetrics} from 'sentry/views/performance/landing/widgets/utils';
import {PerformanceWidgetSetting} from 'sentry/views/performance/landing/widgets/widgetDefinitions';
import Onboarding from 'sentry/views/performance/onboarding';
import Table from 'sentry/views/performance/table';
Expand Down Expand Up @@ -73,6 +77,7 @@ function MobileOverviewPage() {
const {projects} = useProjects();
const onboardingProject = useOnboardingProject();
const navigate = useNavigate();
const mepSetting = useMEPSettingContext();

const withStaticFilters = canUseMetricsData(organization);

Expand All @@ -94,15 +99,19 @@ function MobileOverviewPage() {
PerformanceWidgetSetting.MOST_SLOW_FRAMES,
PerformanceWidgetSetting.MOST_FROZEN_FRAMES,
];
const tripleChartRowCharts = [
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
];
const tripleChartRowCharts = filterAllowedChartsMetrics(
organization,
[
PerformanceWidgetSetting.TPM_AREA,
PerformanceWidgetSetting.DURATION_HISTOGRAM,
PerformanceWidgetSetting.P50_DURATION_AREA,
PerformanceWidgetSetting.P75_DURATION_AREA,
PerformanceWidgetSetting.P95_DURATION_AREA,
PerformanceWidgetSetting.P99_DURATION_AREA,
PerformanceWidgetSetting.FAILURE_RATE_AREA,
],
mepSetting
);

if (organization.features.includes('mobile-vitals')) {
columnTitles = [...columnTitles.slice(0, 5), 'ttid', ...columnTitles.slice(5, 0)];
Expand Down

0 comments on commit f6a4b24

Please sign in to comment.