Skip to content

Benchmark CI#1394

Closed
dmitryglhf wants to merge 2 commits intomasterfrom
benchmark-ci
Closed

Benchmark CI#1394
dmitryglhf wants to merge 2 commits intomasterfrom
benchmark-ci

Conversation

@dmitryglhf
Copy link
Copy Markdown
Collaborator

@dmitryglhf dmitryglhf commented Jun 18, 2025

This is a 🙋 feature or enhancement.

  • Added AutoMLBenchmark to workflows

@dmitryglhf dmitryglhf changed the title Benchmark ci Benchmark CI Jun 18, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Code has no PEP8 errors!

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.75%. Comparing base (0cd7f05) to head (ae98cae).
⚠️ Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1394      +/-   ##
==========================================
- Coverage   80.77%   80.75%   -0.02%     
==========================================
  Files         148      148              
  Lines       10755    10755              
==========================================
- Hits         8687     8685       -2     
- Misses       2068     2070       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmitryglhf
Copy link
Copy Markdown
Collaborator Author

dmitryglhf commented Jun 18, 2025

Столкнулся с ограничениями, когда тестировал в форке, что в github actions доступно только 4 ядра. Это ограничение распространяется на основной репозиторий? Если да, то нужен ли такой workflow?

@dmitryglhf dmitryglhf requested a review from nicl-nno June 18, 2025 00:24
@nicl-nno
Copy link
Copy Markdown
Collaborator

Столкнулся с ограничениями, когда тестировал в форке, что в github actions доступно только 4 ядра. Это ограничение распространяется на основной репозиторий? Если да, то нужен ли такой workflow?

Для удобной проверки можно его оставить, но точно не в полном объеме - у гитхаба наверняка его какое-то ограничение по CPU-часам на бесплатной учетке. Мб на каком-то конкретных датасетах запускать c небольшим таймаутом.

@dmitryglhf
Copy link
Copy Markdown
Collaborator Author

Столкнулся с ограничениями, когда тестировал в форке, что в github actions доступно только 4 ядра. Это ограничение распространяется на основной репозиторий? Если да, то нужен ли такой workflow?

Для удобной проверки можно его оставить, но точно не в полном объеме - у гитхаба наверняка его какое-то ограничение по CPU-часам на бесплатной учетке. Мб на каком-то конкретных датасетах запускать c небольшим таймаутом.

А через actions есть возможность безопасно подключаться к нашему серверу?

@nicl-nno
Copy link
Copy Markdown
Collaborator

Техническая есть, но с точки зрения безопасности не хотят выставлять их наружу для такого использования. @Nunkyl с этим разбиралась.

@nicl-nno
Copy link
Copy Markdown
Collaborator

Что тогда с этим PR-ом? Можно наверное закомментаровать всё кроме test-пресета, чтобы не исчерпать лимиты гитхаба?

@dmitryglhf
Copy link
Copy Markdown
Collaborator Author

Что тогда с этим PR-ом? Можно наверное закомментаровать всё кроме test-пресета, чтобы не исчерпать лимиты гитхаба?

Вообще я думал, как будет время, подробнее разобраться с этим. Хотел посмотреть может быть в каких-то сервисах будет возможность это недорого запускать. Пока, действительно, можно оставить только test-пресет.

@dmitryglhf dmitryglhf closed this Nov 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants