Data Explorer: Causes program crash when displaying small and large numbers

[DavidMatthews83]

System details:

Positron and OS details:

Positron Version: 2024.12.0 (system setup) build 96
Code - OSS Version: 1.93.0
Commit: c5ce275
Date: 2024-11-28T02:50:45.229Z
Electron: 30.4.0
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: 12.4.254.20-electron.0
OS: Windows_NT x64 10.0.19045

Interpreter details:

R 4.3.0

Describe the issue:

Positron crashes using R when trying to view data in the data explorer when there is a large variation in the numbers being displayed

Steps to reproduce the issue:

small_numbers <- data.frame(
Values = as.integer(runif(5000, min = 1, max = 200))
)
#View(small_numbers) #displays ok in Data Explorer

big_numbers <- data.frame(
Values = as.integer(runif(10, min = 1e7, max = 1e8))
)
#View(big_numbers) #displays ok in Data Explorer

big_and_small_numbers <- rbind(small_numbers,big_numbers)

View(big_and_small_numbers) #crashes positron

#having less small numbers and binding to the large works but is very slow eg.
small_numbers <- data.frame(
Values = as.integer(runif(100, min = 1, max = 200))
)

Expected or desired behavior:

big_and_small_numbers should display in the data explorer

Were there any error messages in the UI, Output panel, or Developer Tools console?

I get this from the developer tools

ERR RPC timed out after 5 seconds: {"jsonrpc":"2.0","method":"get_row_labels","params":{"selection":{"first_index":0,"last_index":84},"format_options":{"large_num_digits":2,"small_num_digits":4,"max_integral_digits":7,"max_value_length":1000,"thousands_sep":""}}}
workbench.desktop.main.js:372468 Dropping event 'GetRowLabelsReply' on comm a1bcc6d5-341c-4d88-9715-07fd161b234e: undefined (No listeners for event 'GetRowLabelsReply'

And this

[dfalbel]

The problem is that

x <- as.integer(c(runif(10, min = 1e7, max = 1e8), runif(5000, min = 1, max = 200)))
grDevices::nclass.FD(x)
#> [1] 7244428

Causing the backend to send a huge amount of data to the front-end when computing the column profile.
We'll probably want to hard code a limit for them number of classes.

Thanks for reporting @DavidMatthews83

[testlabauto]

Verified Fixed

Positron Version(s) : 2025.01.0-124
OS Version          : OSX

Test scenario(s)

Verified with initial filing instructions.

Link(s) to TestRail test cases run or created: