Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc text tasks movement on water #2076

Open
wants to merge 4 commits into
base: devel
Choose a base branch
from

Conversation

SugarHedgehog
Copy link
Collaborator

No description provided.

zdn/fast_set.sh Outdated
@@ -6,7 +6,21 @@
#относительный путь до скрипта и до обрабатываемой папки
#../fast_set.sh ../../matege2023p/4

array=()
array=(
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А это точно стОит сливать в основную ветку?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Скорее нет, чем да

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Но каким-то образом нужно сохранять список

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класть этот список каким-то образом в папку с набором?..

@@ -1,4 +1,19 @@
(function() {
let key = "7";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не-не-не, такое значение ключа не пойдёт! Нужно найти на РешуЕГЭ и взять в качестве ключа номер оттуда

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оно так работает
fast set генерирует доп строку в main по номеру шаблона

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

window.nabor.preferences['7'] = ['total_distance'];

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оно не должно работать ТАК. Ключ для preferences должен быть хоть сколь-либо осмысленным. Поэтому либо уговорить его работать не так, либо поправить за ним.

Либо можно просто переименовать шаблон... А везде, где он цепляется симлинком, или в main.js / fipi.js, переименовать. Не самое приятное, но можно. Либо можно переименовать под РешуЕГЭ только в том наборе, из которого работает fast-set.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это всё натягивание совы на глобус
Осмысленным тут должна быть только ['total_distance']
А сейчас это всё приведёт к перелопачиванию огромного количества симлинков

}, {
vel: 'суммарное пройденное лодкой расстояние',
zna: s * 2,
rod: 2,
nah: 1,
nah: (variant == 3),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не уверен, что здесь нужны скобки, но со скобками в принципе тоже годится.

5999,
].iz()
window.comment='5999';
chas2.task.setMinimaxFunctionTask.forbidOpenEnds = true;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это-то тут зачем?!

525375,
].iz()
window.comment='525375';
chas2.task.setMinimaxFunctionTask.forbidOpenEnds = true;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И тут...

Copy link
Owner

@nickkolok nickkolok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В основную ветку принимать надо - иначе техдолг. Но разрезку надо сделать аккуратно!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants