8000 fixed typo in EfficientNet's model variant from v2_ to v2_s by saad-palapa · Pull Request #3628 · ludwig-ai/ludwig · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fixed typo in EfficientNet's model variant from v2_ to v2_s #3628

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

Merged

Conversation

saad-palapa
Copy link
Contributor

Code Pull Requests

Fixed a typo in EfficientNet's model variant that broke training

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 .

@justinxzhao
Copy link
Contributor

Hi @saad-palapa, thanks for the change.

Looks like this is indeed an error, with v2_s as the correct variant name.

@github-actions
Copy link

Unit Test Results

  4 files  ±0    4 suites  ±0   36m 15s ⏱️ + 6m 17s
31 tests ±0  24 ✔️  - 2    7 💤 +2  0 ±0 
62 runs  ±0  48 ✔️  - 4  14 💤 +4  0 ±0 

Results for commit 1f0aa00. ± Comparison against base commit 42723e3.

This pull request skips 2 tests.
tests.regression_tests.benchmark.test_model_performance ‑ test_performance[ames_housing.gbm.yaml]
tests.regression_tests.benchmark.test_model_performance ‑ test_performance[mercedes_benz_greener.gbm.yaml]

@saad-palapa saad-palapa force-pushed the fix-efficientnet-model-variant-typo branch from 1f0aa00 to 2032f0b Compare September 18, 2023 17:01
@justinxzhao justinxzhao merged commit 86d2d61 into ludwig-ai:master Sep 18, 2023
Infernaught pushed a commit to Infernaught/nightlyfix that referenced this pull request Sep 19, 2023
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
0