-
Notifications
You must be signed in to change notification settings - Fork 856
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
[tabular, timeseries] Rename cached version file to version.txt
from __version__
#4203
Conversation
version.txt
from __version__
version.txt
from __version__
|
Job PR-4203-c5fca48 is done. |
@@ -4152,6 +4152,16 @@ def _load_version_file(cls, path) -> str: | |||
version = load_str.load(path=version_file_path) | |||
return version | |||
|
|||
@classmethod | |||
def _load_version_file_old(cls, path) -> str: |
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.
Maybe we could add the logic of trying both old & new filenames to _load_version_file
instead of introducing a new method _load_version_file_old
? I feel like the namespace of Predictor
already contains quite a few methods.
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.
fair point, updated
|
Job PR-4203-f720197 is done. |
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.
LGTM!
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.
LGTM, thanks!
Issue #, if available:
Resolves #4161
Description of changes:
__version__
that contained the AutoGluon version as a string at the time of saving. However, Kaggle doesn't allow files named__version__
to be uploaded for an unknown reason.version.txt
, while keeping backwards compatibility.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.