Skip to content

<DO_NOT_MERGE>[TROUBLESHOOTING][BUGFIX]Empty test commit (in order to make pull request) so as to run tests.</DO_NOT_MERGE>#3973

Closed
alexsherstinsky wants to merge 3 commits into
masterfrom
bugfix/alexsherstinsky/integration_tests/fix_failing_experiment_and_gbm_and_preprocessing_tests-2024_03_21-34
Closed

<DO_NOT_MERGE>[TROUBLESHOOTING][BUGFIX]Empty test commit (in order to make pull request) so as to run tests.</DO_NOT_MERGE>#3973
alexsherstinsky wants to merge 3 commits into
masterfrom
bugfix/alexsherstinsky/integration_tests/fix_failing_experiment_and_gbm_and_preprocessing_tests-2024_03_21-34

Conversation

@alexsherstinsky

Copy link
Copy Markdown
Collaborator

Scope

Troubleshooting multiple test failures.

Code Pull Requests

Please provide the following:

  • a clear explanation of what your code does
  • if applicable, a reference to an issue
  • a reproducible test for your PR (code, config and data sample)

Documentation Pull Requests

Note that the documentation HTML files are in docs/ while the Markdown sources are in mkdocs/docs.

If you are proposing a modification to the documentation you should change only the Markdown files.

api.md is automatically generated from the docstrings in the code, so if you want to change something in that file, first modify ludwig/api.py docstring, then run mkdocs/code_docs_autogen.py, which will create mkdocs/docs/api.md .

@github-actions

github-actions Bot commented Mar 21, 2024

Copy link
Copy Markdown

Unit Test Results

  6 files  ±0    6 suites  ±0   14m 22s ⏱️ +2s
12 tests ±0    9 ✔️ ±0    3 💤 ±0  0 ±0 
60 runs  ±0  42 ✔️ ±0  18 💤 ±0  0 ±0 

Results for commit 79454c4. ± Comparison against base commit 337f407.

♻️ This comment has been updated with latest results.

…ix_failing_experiment_and_gbm_and_preprocessing_tests-2024_03_21-34
…ix_failing_experiment_and_gbm_and_preprocessing_tests-2024_03_21-34
@mhabedank

Copy link
Copy Markdown
Collaborator

hey @alexsherstinsky is this still needed or can I close it?

@w4nderlust

Copy link
Copy Markdown
Collaborator

Closing this as it was marked DO NOT MERGE and was an empty troubleshooting commit from March 2024. The test failures it was investigating have since been resolved in #4059 (which also removes the GBM subsystem entirely).

@w4nderlust w4nderlust closed this Feb 27, 2026
@w4nderlust
w4nderlust deleted the bugfix/alexsherstinsky/integration_tests/fix_failing_experiment_and_gbm_and_preprocessing_tests-2024_03_21-34 branch February 27, 2026 10:50
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.

3 participants