plotly / plotly/plotly.R

crosstalk equality

Đang mở
#1,167 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Ngôn ngữ chính
R
Star
2.7k
Fork
641
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

I discovered this as I worked through an example of linking leaflet with DT and plotly through crosstalk (example). It seems plotly converts crosstalk keys to String and then uses exact equality match .indexOf (see lines). Thinking through how best to solve this, I came up with the following:

  1. Not convert crosstalk key to string in plotly, but I am guessing there was a reason to do the conversion for plotly traces.

  2. Perform match using something other than indexOf or ===. For the purposes of my example I used [].filter, but this is inefficient.

  3. Expect other htmlwidget authors to also convert to string, or add both the string and non-string forms to crosstalk select.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Tái hiện ví dụ crosstalk được liên kết trong experiments/select_crosstalk.R và kiểm tra logic đối sánh được tham chiếu tại dòng 739 của plotly.js. So sánh các biểu diễn khóa được leaflet, DT và plotly sử dụng, sau đó xác định hành vi đối sánh nào cần nhất quán trong toàn bộ ví dụ trước khi thay đổi bất kỳ điều gì. Hoàn tất khi các widget được liên kết chọn cùng một khóa một cách đáng tin cậy.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
javascript, r
Lĩnh vực
data-visualization
Loại issue
Lỗi
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.