Add installed_extensions information #2018
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Fixes #1974 |
romain-intel
force-pushed
the
fix/add-ext-info-to-env
branch
from
9418922
to
5984363
Compare
This adds information to the system tags about the metaflow extensions that are installed. This makes it easier to reproduce the exact same environment. Information about whether the package information is complete or not is also included. This PR also fixes some issues with extension loading and allows extension information to be gathered in a programatic fashion thereby enabling the discovery of extensions.
romain-intel
force-pushed
the
fix/add-ext-info-to-env
branch
from
5984363
to
eec9099
Compare
romain-intel
added a commit
that referenced
this pull request
Sep 16, 2024
* Modify getting Metaflow version Favors reading the INFO file if present to be able to have the most accurate version of Metaflow when executing remotely (especially in the presence of extensions). Also limit reading the INFO file to once per process (as opposed to possibly twice). Finally, gets the version of the source of Metaflow (and not the current directory) * Addressed comments * Add support for overriding version string * Fix issue with brew (hopefully) * Revert "Add support for overriding version string" This reverts commit 0434e97. * Add installed_extensions information (#2018) * Add installed_extensions information This adds information to the system tags about the metaflow extensions that are installed. This makes it easier to reproduce the exact same environment. Information about whether the package information is complete or not is also included. This PR also fixes some issues with extension loading and allows extension information to be gathered in a programatic fashion thereby enabling the discovery of extensions. * Allow users to configure where extensions are searched * Move EXTENSIONS_SEARCH_DIRS out of config * Move the extension installation information to _graph_info * Use .metadata[X] for name and version since not all importlibs have the aliases
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds information to the system tags about the metaflow extensions that are installed. This makes it easier to reproduce the exact same environment. Information about whether the package information is complete or not is also included.
This PR also fixes some issues with extension loading and allows extension information to be gathered in a programatic fashion thereby enabling the discovery of extensions.