Remove attrdict dependency as it doesn't work for Python >3.9#126
Open
kks32 wants to merge 2 commits intoTACC:masterfrom
Open
Remove attrdict dependency as it doesn't work for Python >3.9#126kks32 wants to merge 2 commits intoTACC:masterfrom
kks32 wants to merge 2 commits intoTACC:masterfrom
Conversation
AttrDict is no longer available and this code breaks for Python > 3.9
|
Hello Krishna, Since Tapis V2 is on track to be completely deprecated, it's unlikely that there will be another official release on PyPI. If you need to use AgavePy with newer versions of Python, the suggested workflow is to use DesignSafe's fork of AttrDict which updates the stdlib imports for Python 3.10+ compatiblity . You can pip install it using Or add the following line to your This fork has been deployed in the DesignSafe portal for several months now with no issue, using Python 3.11. |
This file contains hidden or 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
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.
Description
This PR uses a local custom version of AttrDict already defined in util instead of attrdict, which is deprecated and breaks for Python >3.9
Motivation and Context
AgavePy can run on Python >3.9
How Has This Been Tested?
Types of changes
attrdictis still used in one place. #125Checklist:
git commit -s