Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Improve error handling to indicate cause of error and if overall push worked or not, especially with "Cannot distribute outlines error" for panel pushes

Đang mở
  • Có gì không ổn?
#212 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

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

Hướng nghiên cứu

Bắt đầu với luồng push gọi DistributeOutlines cho các tầng rồi áp dụng các thiết lập fragment. Tái hiện chuỗi bật/tắt trong Structural base push test script được liên kết và so sánh kết quả được báo cáo khi việc phân phối outline hoặc thiết lập fragment thất bại với các phần tử thực sự đã được push. Hoàn thành khi các lỗi xác định nguyên nhân, cho biết push tổng thể có thành công hay không và cung cấp thông tin khắc phục.

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

Mô tả

type:bug
Description:

Currently an opaque error is returned if pushing, and then trying to push again. This error is due to the inability to do a repeated push in this way, and should be handled with a reported error accordingly. We should also have clear error handling for floors, as they currently report an error when the DistributeOutlines method fails for the collection of floors on a single story. When this occurs, the fragment setting can also fail, throwing a similarly opaque error that implies the push failed when all other elements actually still pushed correctly.

Steps to reproduce:

Toggle push to true, false, and true again when pushing levels and other elements. Our standard structural test script includes floors that trigger the distribute outlines failure described above as well, which then returns the mentioned opaque error.

Expected behaviour:

Errors clearly indicating what went wrong, and what still worked, along with info needed to resolve the error.

Test file(s):

Structural base push test script.

Ngôn ngữ chính
C#
Star
5
Fork
0
Merge trung bình
5 ngày 8 giờ
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

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.

Issue khác của BHoM/RAM_Toolkit

Tất cả issue của BHoM/RAM_Toolkit

Issue tương tự

Thêm issue về C#

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.