Conversation
Artem-Goldenberg
left a comment
Artem-Goldenberg
left a comment
There was a problem hiding this comment.
Написал какие то вопросы, но они не очень важные, так что если все работает то можешь так заливать, или можешь что то подправить но если времени нет то пофиг
client/templates/main.html
Outdated
| loss_type_selector.addEventListener("change", () => { | ||
| setModelView("irrelevant") | ||
| const parameters = getParametersFromNode(last_selected_node_id) | ||
| console.log("last id: " + last_selected_layer_id) |
There was a problem hiding this comment.
Здесь принт нужен?
client/templates/main.html
Outdated
| setModelView("irrelevant") | ||
| const parameters = getParametersFromNode(last_selected_node_id) | ||
| console.log("last id: " + last_selected_layer_id) | ||
| const new_value = document.getElementById("loss-type-parameter").value |
There was a problem hiding this comment.
Здесь можно сразу loss_type_selector.value
client/templates/main.html
Outdated
| if (new_value == "MSELoss") { | ||
| parameters[i][1] = "MSELoss" | ||
| } else { | ||
| parameters[i][1] = "CrossEntropyLoss" | ||
| } |
There was a problem hiding this comment.
А нельзя тут сразу присвоить new_value без if?
client/templates/main.html
Outdated
| try { | ||
| let ObjData = { | ||
| type: "Data", | ||
| parameters: "inputs=0;outputs=1;width=2", |
There was a problem hiding this comment.
О хорошо, меньше мусорка
| case "special": | ||
| if (layer.name == "Loss") { | ||
| return document.querySelector("#loss-inputs-sortable-list") | ||
| } |
There was a problem hiding this comment.
А здесь layer.class нельзя сделать loss?
There was a problem hiding this comment.
Я изначально так и сделал, но потом вернул обратно. Если бы я так и оставил класс loss, то надо было еще в нескольких методах дописывать условия всякие. Если раньше метод проверял, что класс слоя special (data, output, target, loss), то теперь ему бы пришлось проверять еще и класс loss.
client/templates/main.html
Outdated
| document.querySelector("#linear-layer-parameters").style.display = "block" | ||
| function setupLayerParameters(layer, layer_id) { | ||
| let layer_parameters = getParametersFromNode(layer_id) | ||
| console.log(layer_parameters) |
There was a problem hiding this comment.
Наверно здесь можно убрать
| for (i in layer_parameters) { | ||
| if (layer_parameters[i][0] == "bias") { | ||
| document.querySelector("#bias-parameter").value = layer_parameters[i][1] == "1" | ||
| } | ||
| if (layer_parameters[i][0] == "inFeatures") { | ||
| document.querySelector("#linear-inFeatures-parameter").value = layer_parameters[i][1] | ||
| } | ||
| if (layer_parameters[i][0] == "outFeatures") { | ||
| document.querySelector("#linear-outFeatures-parameter").value = layer_parameters[i][1] | ||
| } |
There was a problem hiding this comment.
Ну здесь наверное везде тоже можно switch сделать
3 participants
ID-154
Просто добавил селектор в параметры слоя Loss. Для этого пришлось сделать слою Loss отдельный класс (при создании слоя Loss в параметрах теперь не
special, аloss), чтоб корректно отображались параметры именно для этого слоя.Возможно, могли пострадать какие-то функции из-за перехода от special к loss. Если да, то сообщите.