-
Notifications
You must be signed in to change notification settings - Fork 456
chore: explicit optionals everywhere #14535
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
Conversation
|
Bootstrap import analysisComparison of import times between this PR and base. SummaryThe average import time from this PR is: 267 ± 2 ms. The average import time from base is: 273 ± 4 ms. The import time difference between this PR and base is: -5.4 ± 0.1 ms. Import time breakdownThe following import paths have grown:
|
Performance SLOsComparing candidate gnufede/mypy-remove-implicit-optional (d3aa4b5) with baseline main (f16de5c) 📈 Performance Regressions (2 suites)📈 iastaspects - 118/118✅ add_aspectTime: ✅ 0.329µs (SLO: <10.000µs 📉 -96.7%) vs baseline: +0.7% Memory: ✅ 37.375MB (SLO: <39.000MB -4.2%) vs baseline: +4.7% ✅ add_inplace_aspectTime: ✅ 0.331µs (SLO: <10.000µs 📉 -96.7%) vs baseline: -0.3% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ add_inplace_noaspectTime: ✅ 0.319µs (SLO: <10.000µs 📉 -96.8%) vs baseline: -0.3% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ add_noaspectTime: ✅ 0.281µs (SLO: <10.000µs 📉 -97.2%) vs baseline: +1.5% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.7% ✅ bytearray_aspectTime: ✅ 1.326µs (SLO: <10.000µs 📉 -86.7%) vs baseline: -0.9% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ bytearray_extend_aspectTime: ✅ 1.377µs (SLO: <10.000µs 📉 -86.2%) vs baseline: -1.0% Memory: ✅ 37.375MB (SLO: <39.000MB -4.2%) vs baseline: +4.8% ✅ bytearray_extend_noaspectTime: ✅ 0.611µs (SLO: <10.000µs 📉 -93.9%) vs baseline: +0.5% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ bytearray_noaspectTime: ✅ 0.484µs (SLO: <10.000µs 📉 -95.2%) vs baseline: ~same Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ bytes_aspectTime: ✅ 1.288µs (SLO: <10.000µs 📉 -87.1%) vs baseline: +1.0% Memory: ✅ 37.356MB (SLO: <39.000MB -4.2%) vs baseline: +4.7% ✅ bytes_noaspectTime: ✅ 0.494µs (SLO: <10.000µs 📉 -95.1%) vs baseline: +1.6% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ bytesio_aspectTime: ✅ 1.342µs (SLO: <10.000µs 📉 -86.6%) vs baseline: -0.3% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ bytesio_noaspectTime: ✅ 0.497µs (SLO: <10.000µs 📉 -95.0%) vs baseline: +0.7% Memory: ✅ 37.473MB (SLO: <39.000MB -3.9%) vs baseline: +5.1% ✅ capitalize_aspectTime: ✅ 0.730µs (SLO: <10.000µs 📉 -92.7%) vs baseline: +0.4% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.8% ✅ capitalize_noaspectTime: ✅ 0.433µs (SLO: <10.000µs 📉 -95.7%) vs baseline: -0.2% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.7% ✅ casefold_aspectTime: ✅ 0.737µs (SLO: <10.000µs 📉 -92.6%) vs baseline: +0.5% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ casefold_noaspectTime: ✅ 0.369µs (SLO: <10.000µs 📉 -96.3%) vs baseline: +0.7% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ decode_aspectTime: ✅ 0.726µs (SLO: <10.000µs 📉 -92.7%) vs baseline: +1.0% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.8% ✅ decode_noaspectTime: ✅ 0.422µs (SLO: <10.000µs 📉 -95.8%) vs baseline: +1.5% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ encode_aspectTime: ✅ 0.710µs (SLO: <10.000µs 📉 -92.9%) vs baseline: -0.6% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.8% ✅ encode_noaspectTime: ✅ 0.400µs (SLO: <10.000µs 📉 -96.0%) vs baseline: +0.3% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ format_aspectTime: ✅ 3.348µs (SLO: <10.000µs 📉 -66.5%) vs baseline: -0.5% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ format_map_aspectTime: ✅ 3.545µs (SLO: <10.000µs 📉 -64.5%) vs baseline: 📈 +12.8% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.1% ✅ format_map_noaspectTime: ✅ 0.775µs (SLO: <10.000µs 📉 -92.2%) vs baseline: -0.9% Memory: ✅ 37.493MB (SLO: <39.000MB -3.9%) vs baseline: +5.2% ✅ format_noaspectTime: ✅ 0.595µs (SLO: <10.000µs 📉 -94.1%) vs baseline: +0.6% Memory: ✅ 37.356MB (SLO: <39.000MB -4.2%) vs baseline: +4.6% ✅ index_aspectTime: ✅ 0.346µs (SLO: <10.000µs 📉 -96.5%) vs baseline: -0.6% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ index_noaspectTime: ✅ 0.281µs (SLO: <10.000µs 📉 -97.2%) vs baseline: +1.9% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +4.9% ✅ join_aspectTime: ✅ 1.248µs (SLO: <10.000µs 📉 -87.5%) vs baseline: +1.2% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ join_noaspectTime: ✅ 0.489µs (SLO: <10.000µs 📉 -95.1%) vs baseline: +0.4% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ ljust_aspectTime: ✅ 10.698µs (SLO: <20.000µs 📉 -46.5%) vs baseline: +0.3% Memory: ✅ 37.749MB (SLO: <39.000MB -3.2%) vs baseline: +5.1% ✅ ljust_noaspectTime: ✅ 0.407µs (SLO: <10.000µs 📉 -95.9%) vs baseline: +0.2% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.6% ✅ lower_aspectTime: ✅ 2.236µs (SLO: <10.000µs 📉 -77.6%) vs baseline: ~same Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.7% ✅ lower_noaspectTime: ✅ 0.365µs (SLO: <10.000µs 📉 -96.3%) vs baseline: +0.1% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ lstrip_aspectTime: ✅ 10.875µs (SLO: <20.000µs 📉 -45.6%) vs baseline: +0.4% Memory: ✅ 37.611MB (SLO: <39.000MB -3.6%) vs baseline: +4.9% ✅ lstrip_noaspectTime: ✅ 0.384µs (SLO: <10.000µs 📉 -96.2%) vs baseline: +0.5% Memory: ✅ 37.375MB (SLO: <39.000MB -4.2%) vs baseline: +4.7% ✅ modulo_aspectTime: ✅ 0.933µs (SLO: <10.000µs 📉 -90.7%) vs baseline: -0.3% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ modulo_aspect_for_bytearray_bytearrayTime: ✅ 1.419µs (SLO: <10.000µs 📉 -85.8%) vs baseline: -0.7% Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ modulo_aspect_for_bytesTime: ✅ 0.917µs (SLO: <10.000µs 📉 -90.8%) vs baseline: +0.9% Memory: ✅ 37.375MB (SLO: <39.000MB -4.2%) vs baseline: +4.7% ✅ modulo_aspect_for_bytes_bytearrayTime: ✅ 1.121µs (SLO: <10.000µs 📉 -88.8%) vs baseline: ~same Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ modulo_noaspectTime: ✅ 0.625µs (SLO: <10.000µs 📉 -93.8%) vs baseline: +0.4% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.6% ✅ replace_aspectTime: ✅ 5.073µs (SLO: <10.000µs 📉 -49.3%) vs baseline: 📈 +13.6% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +5.0% ✅ replace_noaspectTime: ✅ 0.467µs (SLO: <10.000µs 📉 -95.3%) vs baseline: +1.2% Memory: ✅ 37.356MB (SLO: <39.000MB -4.2%) vs baseline: +4.5% ✅ repr_aspectTime: ✅ 0.909µs (SLO: <10.000µs 📉 -90.9%) vs baseline: -0.3% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ repr_noaspectTime: ✅ 0.411µs (SLO: <10.000µs 📉 -95.9%) vs baseline: -1.3% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.7% ✅ rstrip_aspectTime: ✅ 11.393µs (SLO: <20.000µs 📉 -43.0%) vs baseline: +5.4% Memory: ✅ 37.631MB (SLO: <39.000MB -3.5%) vs baseline: +4.9% ✅ rstrip_noaspectTime: ✅ 0.380µs (SLO: <10.000µs 📉 -96.2%) vs baseline: ~same Memory: ✅ 37.454MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ slice_aspectTime: ✅ 0.484µs (SLO: <10.000µs 📉 -95.2%) vs baseline: -0.1% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +5.1% ✅ slice_noaspectTime: ✅ 0.442µs (SLO: <10.000µs 📉 -95.6%) vs baseline: -1.0% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +5.0% ✅ stringio_aspectTime: ✅ 1.588µs (SLO: <10.000µs 📉 -84.1%) vs baseline: +1.0% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +4.9% ✅ stringio_noaspectTime: ✅ 0.714µs (SLO: <10.000µs 📉 -92.9%) vs baseline: +0.4% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +4.9% ✅ strip_aspectTime: ✅ 10.999µs (SLO: <20.000µs 📉 -45.0%) vs baseline: +1.1% Memory: ✅ 37.650MB (SLO: <39.000MB -3.5%) vs baseline: +4.9% ✅ strip_noaspectTime: ✅ 0.385µs (SLO: <10.000µs 📉 -96.2%) vs baseline: +0.3% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ swapcase_aspectTime: ✅ 2.450µs (SLO: <10.000µs 📉 -75.5%) vs baseline: -0.5% Memory: ✅ 37.493MB (SLO: <39.000MB -3.9%) vs baseline: +5.1% ✅ swapcase_noaspectTime: ✅ 0.541µs (SLO: <10.000µs 📉 -94.6%) vs baseline: -0.1% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.8% ✅ title_aspectTime: ✅ 2.389µs (SLO: <10.000µs 📉 -76.1%) vs baseline: -0.5% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +4.9% ✅ title_noaspectTime: ✅ 0.504µs (SLO: <10.000µs 📉 -95.0%) vs baseline: -0.6% Memory: ✅ 37.434MB (SLO: <39.000MB -4.0%) vs baseline: +4.9% ✅ translate_aspectTime: ✅ 3.365µs (SLO: <10.000µs 📉 -66.4%) vs baseline: +2.1% Memory: ✅ 37.395MB (SLO: <39.000MB -4.1%) vs baseline: +4.7% ✅ translate_noaspectTime: ✅ 1.037µs (SLO: <10.000µs 📉 -89.6%) vs baseline: -0.7% Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.9% ✅ upper_aspectTime: ✅ 2.267µs (SLO: <10.000µs 📉 -77.3%) vs baseline: ~same Memory: ✅ 37.415MB (SLO: <39.000MB -4.1%) vs baseline: +4.8% ✅ upper_noaspectTime: ✅ 0.375µs (SLO: <10.000µs 📉 -96.2%) vs baseline: +1.8% Memory: ✅ 37.375MB (SLO: <39.000MB -4.2%) vs baseline: +4.8% 📈 telemetryaddmetric - 30/30✅ 1-count-metric-1-timesTime: ✅ 3.505µs (SLO: <20.000µs 📉 -82.5%) vs baseline: 📈 +11.4% Memory: ✅ 31.418MB (SLO: <34.000MB -7.6%) vs baseline: +4.9% ✅ 1-count-metrics-100-timesTime: ✅ 216.151µs (SLO: <250.000µs 📉 -13.5%) vs baseline: +0.9% Memory: ✅ 31.418MB (SLO: <34.000MB -7.6%) vs baseline: +4.5% ✅ 1-distribution-metric-1-timesTime: ✅ 3.372µs (SLO: <20.000µs 📉 -83.1%) vs baseline: 📈 +13.0% Memory: ✅ 31.497MB (SLO: <34.000MB -7.4%) vs baseline: +4.9% ✅ 1-distribution-metrics-100-timesTime: ✅ 192.907µs (SLO: <220.000µs 📉 -12.3%) vs baseline: +0.2% Memory: ✅ 31.379MB (SLO: <34.000MB -7.7%) vs baseline: +4.6% ✅ 1-gauge-metric-1-timesTime: ✅ 2.132µs (SLO: <20.000µs 📉 -89.3%) vs baseline: +1.7% Memory: ✅ 31.457MB (SLO: <34.000MB -7.5%) vs baseline: +5.0% ✅ 1-gauge-metrics-100-timesTime: ✅ 124.669µs (SLO: <150.000µs 📉 -16.9%) vs baseline: -0.1% Memory: ✅ 31.438MB (SLO: <34.000MB -7.5%) vs baseline: +4.9% ✅ 1-rate-metric-1-timesTime: ✅ 3.486µs (SLO: <20.000µs 📉 -82.6%) vs baseline: 📈 +10.9% Memory: ✅ 31.438MB (SLO: <34.000MB -7.5%) vs baseline: +5.1% ✅ 1-rate-metrics-100-timesTime: ✅ 215.216µs (SLO: <250.000µs 📉 -13.9%) vs baseline: +0.5% Memory: ✅ 31.398MB (SLO: <34.000MB -7.7%) vs baseline: +4.5% ✅ 100-count-metrics-100-timesTime: ✅ 22.390ms (SLO: <23.500ms -4.7%) vs baseline: +4.3% Memory: ✅ 31.438MB (SLO: <34.000MB -7.5%) vs baseline: +4.8% ✅ 100-distribution-metrics-100-timesTime: ✅ 2.017ms (SLO: <2.250ms 📉 -10.4%) vs baseline: +1.6% Memory: ✅ 31.438MB (SLO: <34.000MB -7.5%) vs baseline: +4.7% ✅ 100-gauge-metrics-100-timesTime: ✅ 1.268ms (SLO: <1.550ms 📉 -18.2%) vs baseline: -0.1% Memory: ✅ 31.477MB (SLO: <34.000MB -7.4%) vs baseline: +5.2% ✅ 100-rate-metrics-100-timesTime: ✅ 2.292ms (SLO: <2.550ms 📉 -10.1%) vs baseline: +3.1% Memory: ✅ 31.418MB (SLO: <34.000MB -7.6%) vs baseline: +4.5% ✅ flush-1-metricTime: ✅ 4.383µs (SLO: <20.000µs 📉 -78.1%) vs baseline: +3.8% Memory: ✅ 31.831MB (SLO: <34.000MB -6.4%) vs baseline: +5.4% ✅ flush-100-metricsTime: ✅ 183.659µs (SLO: <250.000µs 📉 -26.5%) vs baseline: -0.3% Memory: ✅ 31.850MB (SLO: <34.000MB -6.3%) vs baseline: +5.1% ✅ flush-1000-metricsTime: ✅ 2.205ms (SLO: <2.500ms 📉 -11.8%) vs baseline: -0.5% Memory: ✅ 32.539MB (SLO: <34.500MB -5.7%) vs baseline: +4.9% 🟡 Near SLO Breach (5 suites)🟡 djangosimple - 26/26✅ appsecTime: ✅ 20.501ms (SLO: <22.300ms -8.1%) vs baseline: ~same Memory: ✅ 64.546MB (SLO: <66.000MB -2.2%) vs baseline: +5.0% ✅ exception-replay-enabledTime: ✅ 1.348ms (SLO: <1.450ms -7.0%) vs baseline: +0.4% Memory: ✅ 63.472MB (SLO: <66.000MB -3.8%) vs baseline: +4.7% ✅ iastTime: ✅ 20.503ms (SLO: <22.250ms -7.9%) vs baseline: ~same Memory: ✅ 64.527MB (SLO: <66.000MB -2.2%) vs baseline: +4.9% ✅ profilerTime: ✅ 15.237ms (SLO: <16.550ms -7.9%) vs baseline: +0.4% Memory: ✅ 52.868MB (SLO: <53.500MB 🟡 -1.2%) vs baseline: +4.7% ✅ span-code-originTime: ✅ 26.148ms (SLO: <28.200ms -7.3%) vs baseline: -0.2% Memory: ✅ 66.645MB (SLO: <68.500MB -2.7%) vs baseline: +5.0% ✅ tracerTime: ✅ 20.503ms (SLO: <21.750ms -5.7%) vs baseline: -0.2% Memory: ✅ 64.507MB (SLO: <66.000MB -2.3%) vs baseline: +4.8% ✅ tracer-and-profilerTime: ✅ 22.163ms (SLO: <23.500ms -5.7%) vs baseline: +0.4% Memory: ✅ 65.751MB (SLO: <67.000MB 🟡 -1.9%) vs baseline: +5.0% ✅ tracer-dont-create-db-spansTime: ✅ 19.350ms (SLO: <21.500ms 📉 -10.0%) vs baseline: -0.4% Memory: ✅ 64.458MB (SLO: <66.000MB -2.3%) vs baseline: +4.7% ✅ tracer-nativeTime: ✅ 20.469ms (SLO: <21.750ms -5.9%) vs baseline: ~same Memory: ✅ 65.744MB (SLO: <66.000MB 🟡 -0.4%) vs baseline: +4.8% ✅ tracer-no-cachesTime: ✅ 18.446ms (SLO: <19.650ms -6.1%) vs baseline: -0.3% Memory: ✅ 64.464MB (SLO: <66.000MB -2.3%) vs baseline: +4.8% ✅ tracer-no-databasesTime: ✅ 18.848ms (SLO: <20.100ms -6.2%) vs baseline: -0.3% Memory: ✅ 64.113MB (SLO: <66.000MB -2.9%) vs baseline: +4.9% ✅ tracer-no-middlewareTime: ✅ 20.193ms (SLO: <21.500ms -6.1%) vs baseline: ~same Memory: ✅ 64.507MB (SLO: <66.000MB -2.3%) vs baseline: +4.9% ✅ tracer-no-templatesTime: ✅ 20.356ms (SLO: <22.000ms -7.5%) vs baseline: +0.3% Memory: ✅ 64.484MB (SLO: <66.000MB -2.3%) vs baseline: +4.9% 🟡 errortrackingdjangosimple - 6/6✅ errortracking-enabled-allTime: ✅ 18.058ms (SLO: <19.850ms -9.0%) vs baseline: ~same Memory: ✅ 64.423MB (SLO: <65.500MB 🟡 -1.6%) vs baseline: +4.7% ✅ errortracking-enabled-userTime: ✅ 18.069ms (SLO: <19.400ms -6.9%) vs baseline: ~same Memory: ✅ 64.404MB (SLO: <65.500MB 🟡 -1.7%) vs baseline: +4.8% ✅ tracer-enabledTime: ✅ 18.252ms (SLO: <19.450ms -6.2%) vs baseline: +1.0% Memory: ✅ 64.484MB (SLO: <65.500MB 🟡 -1.6%) vs baseline: +5.0% 🟡 flasksimple - 17/17✅ appsec-getTime: ✅ 4.575ms (SLO: <4.750ms -3.7%) vs baseline: -0.2% Memory: ✅ 62.108MB (SLO: <64.500MB -3.7%) vs baseline: +4.7% ✅ appsec-postTime: ✅ 6.575ms (SLO: <6.750ms -2.6%) vs baseline: ~same Memory: ✅ 62.344MB (SLO: <64.500MB -3.3%) vs baseline: +4.9% ✅ appsec-telemetryTime: ✅ 4.574ms (SLO: <4.750ms -3.7%) vs baseline: ~same Memory: ✅ 62.010MB (SLO: <64.500MB -3.9%) vs baseline: +4.5% ✅ debuggerTime: ✅ 1.855ms (SLO: <2.000ms -7.2%) vs baseline: ~same Memory: ✅ 44.709MB (SLO: <45.000MB 🟡 -0.6%) vs baseline: +4.7% ✅ iast-getTime: ✅ 1.858ms (SLO: <2.000ms -7.1%) vs baseline: -0.2% Memory: ✅ 41.642MB (SLO: <49.000MB 📉 -15.0%) vs baseline: +4.8% ✅ profilerTime: ✅ 1.915ms (SLO: <2.100ms -8.8%) vs baseline: -0.4% Memory: ✅ 44.355MB (SLO: <46.500MB -4.6%) vs baseline: +4.6% ✅ tracerTime: ✅ 3.368ms (SLO: <3.650ms -7.7%) vs baseline: -0.3% Memory: ✅ 51.413MB (SLO: <53.500MB -3.9%) vs baseline: +4.8% ✅ tracer-nativeTime: ✅ 3.372ms (SLO: <3.650ms -7.6%) vs baseline: ~same Memory: ✅ 52.671MB (SLO: <53.500MB 🟡 -1.5%) vs baseline: +4.7% 🟡 flasksqli - 6/6✅ appsec-enabledTime: ✅ 3.945ms (SLO: <4.200ms -6.1%) vs baseline: -0.1% Memory: ✅ 62.374MB (SLO: <66.000MB -5.5%) vs baseline: +5.0% ✅ iast-enabledTime: ✅ 2.523ms (SLO: <2.800ms -9.9%) vs baseline: +0.3% Memory: ✅ 58.168MB (SLO: <59.000MB 🟡 -1.4%) vs baseline: +4.9% ✅ tracer-enabledTime: ✅ 2.078ms (SLO: <2.250ms -7.7%) vs baseline: +0.1% Memory: ✅ 51.263MB (SLO: <53.500MB -4.2%) vs baseline: +4.8% 🟡 otelspan - 22/22✅ add-eventTime: ✅ 45.036ms (SLO: <47.150ms -4.5%) vs baseline: -0.6% Memory: ✅ 44.655MB (SLO: <46.500MB -4.0%) vs baseline: +5.0% ✅ add-metricsTime: ✅ 322.249ms (SLO: <344.800ms -6.5%) vs baseline: +0.7% Memory: ✅ 554.287MB (SLO: <562.000MB 🟡 -1.4%) vs baseline: +5.0% ✅ add-tagsTime: ✅ 293.824ms (SLO: <314.000ms -6.4%) vs baseline: +0.8% Memory: ✅ 554.267MB (SLO: <563.500MB 🟡 -1.6%) vs baseline: +4.9% ✅ get-contextTime: ✅ 82.949ms (SLO: <92.350ms 📉 -10.2%) vs baseline: ~same Memory: ✅ 39.725MB (SLO: <46.500MB 📉 -14.6%) vs baseline: +4.8% ✅ is-recordingTime: ✅ 42.739ms (SLO: <44.500ms -4.0%) vs baseline: -0.4% Memory: ✅ 43.976MB (SLO: <46.500MB -5.4%) vs baseline: +4.7% ✅ record-exceptionTime: ✅ 61.751ms (SLO: <67.650ms -8.7%) vs baseline: ~same Memory: ✅ 40.079MB (SLO: <46.500MB 📉 -13.8%) vs baseline: +5.1% ✅ set-statusTime: ✅ 48.639ms (SLO: <50.400ms -3.5%) vs baseline: -0.8% Memory: ✅ 44.032MB (SLO: <46.500MB -5.3%) vs baseline: +5.0% ✅ startTime: ✅ 41.971ms (SLO: <43.450ms -3.4%) vs baseline: -0.9% Memory: ✅ 44.016MB (SLO: <46.500MB -5.3%) vs baseline: +4.8% ✅ start-finishTime: ✅ 82.100ms (SLO: <88.000ms -6.7%) vs baseline: -0.3% Memory: ✅ 33.935MB (SLO: <46.500MB 📉 -27.0%) vs baseline: +4.6% ✅ start-finish-telemetryTime: ✅ 83.819ms (SLO: <89.000ms -5.8%) vs baseline: +0.2% Memory: ✅ 34.013MB (SLO: <46.500MB 📉 -26.9%) vs baseline: +5.0% ✅ update-nameTime: ✅ 44.036ms (SLO: <45.150ms -2.5%) vs baseline: -0.7% Memory: ✅ 44.369MB (SLO: <46.500MB -4.6%) vs baseline: +4.9%
|
It's really necessary for |
Explicit is better than implicit. |
tests/appsec/integrations/fastapi_tests/test_fastapi_appsec_iast.py
Outdated
Show resolved
Hide resolved
Yeah but https://mypy.readthedocs.io/en/stable/kinds_of_types.html#the-any-type
|
OK, you are right, mypy doesn't complain, I personally prefer Also, that reduces the scope of the PR, so I prefer it. |
Code style: This PR changes implicit optionals to explicit ones (in type annotations)
Read more here: https://github.com/hauntsaninja/no_implicit_optional?tab=readme-ov-file#whats-going-on
Checklist
Reviewer Checklist