biocore/empress

Save extra work of cos/sin calls

開放

#225 建立於 2020年6月29日

在 GitHub 查看貢獻者指南
 (0 則留言) (0 個反應) (0 位負責人)JavaScript (32 個分叉)auto 404
good first issueperformancerefactoring

倉庫指標

星標
 (56 顆星)
PR 合併指標
 (30 天內沒有已合併 PR)

描述

@wasade pointed this out about this JS code -- there are 8 calls to cos or sin within this block, but we could cut this down to just 4 calls by saving the value of each call for later in the block.

From some more digging, this also impacts this JS code for drawing thick lines, as well as this python code.

貢獻者指南

研究方向
查找提到的三個程式碼區塊（empress.js 第404-420行、第643-655行和tree.py 第437-449行）。識別對同一角度的重複cos/sin呼叫，並用單個計算變數替換，重複使用該值。例如，計算一次cos(angle)並賦值給變數，然後多次使用該變數。
技術棧
javascriptpython
領域
backend
議題類型
重構
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 小時
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
JavaScriptPythonbasic trigonometry
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65