-
Notifications
You must be signed in to change notification settings - Fork 88
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
AMLB-specific test case added #1274
base: master
Are you sure you want to change the base?
Conversation
👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app Simply create a new comment in this PR that says: @docu-mentor run and I will start my analysis. I only look at what you changed @docu-mentor run doc/ README.md In this example, I'll have a look at all files contained in the "doc/" |
👋 Hi, I'm @open-code-helper, an LLM-powered GitHub app Simply create a new comment in this PR that says: @open-code-helper run and I will start my analysis. I only look at what you changed |
All PEP8 errors has been fixed, thanks ❤️ Comment last updated at |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1274 +/- ##
==========================================
+ Coverage 79.88% 79.95% +0.07%
==========================================
Files 146 146
Lines 10065 10051 -14
==========================================
- Hits 8040 8036 -4
+ Misses 2025 2015 -10 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не очень понимаю, как этот PR связан с #1273
исправления в get_metrics
не должны были повлиять на IndexError
в AMLB интеграционнике
по поводу AMLB-specific теста:
верно ли понимаю, что проблема пока регистрируется только на пайплайнах с нодой resample?
наверное, стоит преобразовать np.ndarray
в InputData
и с уже верными индексами (пусть даже idx=np.arange(len(x_train)
) скормить пайплайну на resample.
|
||
@pytest.mark.parametrize('initial_assumption, timeout', | ||
[(resample_pipeline, 0.001), | ||
(None, 5.0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не очень понятна идея, для чего здесь implicit initial_assumption=None
?
будет NoneType object is not subscriptable
где-то при попытке логировать
/fix-pep8 |
prepare test data again
tests that get_metrics works correctly when target is defined as a string and test_data doesn't contain it
Co-authored-by: George Lopatenko <[email protected]>
✅ Integration run - https://github.com/aimclub/FEDOT/actions/runs/8644854445 |
Update fedot/api/main.py Co-authored-by: George Lopatenko <[email protected]>
/fix-pep8 |
✅ Integration run - https://github.com/aimclub/FEDOT/actions/runs/8695868602 |
👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app Simply create a new comment in this PR that says: @docu-mentor run and I will start my analysis. I only look at what you changed @docu-mentor run doc/ README.md In this example, I'll have a look at all files contained in the "doc/" |
Error "IndexError: arrays used as indices must be of integer (or boolean) type" is found if final pipeline contains "resample" op