-
Notifications
You must be signed in to change notification settings - Fork 37
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
Некорректное создание общей таблицы производительности для всех фреймворков #363
Comments
@FenixFly приложите пожалуйста артефакт. это проблема в csv или в excel? |
@n-berezina-nn, проблема при генерации xlsx-файла. |
@FenixFly, есть по всей видимости еще одна проблема. Если проводится эксперимент при одинаковых условиях, но отличается всего один входной параметр (увидела это на MXNet, когда добавила включение и отключение символьных вычислений), то результат перезаписывается, поскольку наши конвертеры в HTML и XLSX не учитывают такую возможность в принципе. При последнем изменении конвертера (изменилось регулярное выражение) в заголовке столбца осталось только устройство без параметров, в результате количество столбцов уменьшилось, но вылезла проблема с наличием разных параметров. Последнее изменение я планирую ревертнуть и решить проблему с указанием пачки в параметре input_shape. Других вариантов я не вижу, поскольку у нас реально со всеми библиотеками будет подобная проблема. |
@FenixFly, в #376 пофиксила формирование таблицы для нескольких фреймворков. Есть конечно 2 момента, которые могут не позволить получить хорошую общую таблицу:
|
При создании общей таблицы производительности для всех фреймворков следующие проблемы:
The text was updated successfully, but these errors were encountered: